Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR: Use GitHub Actions to do automatic testing #253

Merged
merged 5 commits into from
Mar 12, 2020
Merged

PR: Use GitHub Actions to do automatic testing #253

merged 5 commits into from
Mar 12, 2020

Conversation

jitseniesen
Copy link
Member

@jitseniesen jitseniesen commented Mar 12, 2020

This PR switches from CircleCI to GitHub Actions for automatic testing.

It adds automatic testing on Windows and MacOS. One of the tests hung on these OSes, so that test is skipped. Tests are also not run for Python 2.7 under Windows, because pytest segfaults there. I think it is not essential to fix these issues, given that we did not have testing on Windows and MacOS before, and Python 2.7 is end of line.

Thanks to @goanpeca for getting this to work on spyder-kernels which was a good start for me.

Fixes #241

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@59b1e7f). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #253   +/-   ##
=========================================
  Coverage          ?   74.18%           
=========================================
  Files             ?        8           
  Lines             ?      550           
  Branches          ?        0           
=========================================
  Hits              ?      408           
  Misses            ?      142           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 59b1e7f...d9baa26. Read the comment docs.

@jitseniesen jitseniesen changed the title Test actions PR: Use GitHub Actions to do automatic testing Mar 12, 2020
@jitseniesen jitseniesen self-assigned this Mar 12, 2020
@jitseniesen jitseniesen requested a review from goanpeca March 12, 2020 18:39
Copy link
Member

@goanpeca goanpeca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome @jitseniesen ! thanks for working on this!

Looks good to me :-)

@jitseniesen jitseniesen merged commit 5a0ad4f into master Mar 12, 2020
@jitseniesen jitseniesen deleted the actions branch March 12, 2020 21:47
@ccordoba12
Copy link
Member

This is really nice, great work @jitseniesen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move CI to github actions
4 participants