-
Notifications
You must be signed in to change notification settings - Fork 532
Cannot make a PR with requirement of running make check-before-commit
which crashes
#2946
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
Just use |
Thanks for the feedback Chris. I actually had
|
I have no errors when importing "manually"
|
Ah. It looks like you don't have |
Ah I see, apologies for the oversight. The remaining errors seem to be due to different
Due to the long output, click the arrow below to see the whole output.
|
Yeah, there's an open issue (#2791) for supporting nilearn > 0.5. They made some changes that nobody has had time to look into deeply, and so using the There is related discussion in: nilearn/nilearn#1891, nilearn/nilearn#1892 |
I see, then definitely no Should i leave the issue open until |
Summary
I forked the NiPype repo and followed the requirement of running
make check-before-commit
before submitting a PR but I got the output below ending with an error.I am in a conda environment with Python 3.7 but somehow there are some references to python 2.7.
Due to the long output, click the arrow below to see the whole output.
Actual behavior
Error when running make check-before-commit
Expected behavior
No errors
How to replicate the behavior
Try to run
make check-before-commit
on fresh install of nipypeScript/Workflow details
make check-before-commit
Platform details:
Execution environment
Choose one
The text was updated successfully, but these errors were encountered: