-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
BLD: Github Actions #363
BLD: Github Actions #363
Conversation
What does that do? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gerrymanoim ! Comments inline.
fail-fast: false | ||
matrix: | ||
os: [ubuntu-latest] | ||
python-version: [2.7, 3.5, 3.7, 3.8] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this python-version
required if we're explicit below in include
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes - the include is on the version specified I believe. https://help.github.com/en/actions/reference/workflow-syntax-for-github-actions#example-including-configurations-in-a-matrix-build. My understanding is that you're augmenting and existing build.
This PR transitions us from Travis to Github actions. It also does the work to fix flake8 issues and get tests passing on python 2.7 and python 3.5.
Tests still fail in python 3.7/3.8 with newer pandas/numpy/etc.
Future work involves