-
Notifications
You must be signed in to change notification settings - Fork 32
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
[MAINT] run test and flake8 github actions #131
Conversation
Remi-Gau
commented
Aug 30, 2023
•
edited
Loading
edited
- upload coverage report
- make sure tests pass
- rm travis.yml
- add GHA workflow
- add test data instead of downloading from neurovault to minimize failures
- add pre-commit to run flake8 locally upon commit
- closes run tests on github action instead of travis #125, run flake8 via github action instead of travis #126
- closes fix broken badge in readme #129
not sure why we are getting this: https://github.com/miykael/atlasreader/actions/runs/6030134696
|
Will let this rest for today. Finishing fixing this may require diving into libraries I am not familiar with. |
I was able to make the tests pass, and updated the faulty badges. But now it crashes do to some coveralls issue. Unfortunately I'm not familiar with coveralls or its github-actions. I registered on coveralls.io and added the a token via the command line, but it's still failing. Any idea how to debug this? |
the action was fine I just had forgotten to generate the coverage report. 🙈 BTW: added python 3.12 in CI |
Oh super! Thank you for all of this! Looks good to me, will go ahead and merge it! |