-
Notifications
You must be signed in to change notification settings - Fork 62
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
Add github actions CI #487
Conversation
* Streamline matrix to avoid old special cases * Skip archiving flake8 output, because it is now in stdout * Skip archiving pytest output, because it appears in stdout * Temporarily run on pushes to any branch
Okay, I believe I've addressed the comments above. The tests passed here. |
Why is the doc build failed? Other than this and the need to resolve the conflict, all LGTM. |
Looks like we were one of the lucky few to hit a brownout requiring the new .readthedocs.yaml format. I'll try to update it. |
`pip` is now only on the user path, not the system path
As everything looks good to me, and I wish to polish CI and bump Python etc. Merging this as the foundation. Thanks! |
This PR attempts to replicate the CircleCI workflow (which is not working as of this writing) in github actions. It also removes references to CircleCI.
It also fixes a few minor things I needed to do to get the tests to pass:
getargspec
withgetfullargspec
ingalgebra/lt.py
self.nargs = 0
in the branch following thegetargspec
usage, so that it's not referring to a non-existent variableif
statement ingalgebra/mv.py
so the linter doesn't chokepython_requires
syntax insetup.py
This is one half of the split version of #486. The other half is a branch on my fork https://github.com/moble/galgebra/tree/jupyterlite, which starts relative to this PR's first commit; I'll make it into a PR once this is merged.