-
Notifications
You must be signed in to change notification settings - Fork 532
Reconsider CI management #3565
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
Comments
Yeah, I now regret the I agree we can drop the venvs now. It used to be useful to avoid accidentally depending on your build tools in your test environment. |
Now that I am getting very familiar with Hatch, I could give it a go if I have got your green light for it? |
Have you found integrating Hatch with GHA nice? https://github.com/ymyzk/tox-gh-actions makes working with Tox trivial. That said, I'm okay with it in either direction. I just wouldn't spend a ton of time on any maintenance tasks in Nipype 1, as it really can't outlast Python 3.11 without a significant effort that would be better directed toward pydra. |
It's basically two steps: Provisioning and configuring the runtime is handled by |
If hatch works smoothly, that's fine with me. |
It looks like issues with CI started to surface on the last two merged PRs. CI complains that jobs no longer find
black
orpytest
although they appear in the install step in the logs. The only reason I could think of is that the venv is no longer visible or activated between calls to the CI scripts (perhaps as a result of a change in GitHub Actions behaviour)?Perhaps we could refactor CI to not use a dedicated venv? Maybe there are good reasons for it, I have just never seen it done in practice with GHA since each job is executed in a separate throwaway container anyway. It might also make things considerably simpler, as I find the current flow of CI scripts quite hard to follow. Let me know what you think.
Originally posted by @ghisvail in #3529 (comment)
The text was updated successfully, but these errors were encountered: