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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix files removed via make clean #789
Fix files removed via make clean #789
Conversation
Current coverage is 100% (diff: 100%)@@ master #789 diff @@
====================================
Files 15 15
Lines 601 601
Methods 0 0
Messages 0 0
Branches 0 0
====================================
Hits 601 601
Misses 0 0
Partials 0 0
|
Anyone? 馃榿 |
print('pre_gen_project.py~') | ||
""" | ||
) | ||
pre_gen_hook_file = hooks_dir / 'pre_gen_project.py~' |
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.
These files should be in a hooks
directory?
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.
Good catch!
In the long run we probably want to change the code to accept a path rather than using the current working directory + hooks
.
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.
Yeah 馃憤
Should be fixed now. 馃槃 |
馃憤 |
Thank you for the review @michaeljoseph! 馃檱 |
We've added a couple of hook backup files as static files to our tests to make sure that they are not accidentally picked up as hooks in templates, see #768.
The problem is that
make clean
removes such files and I have togit reset --hard
to restore them as of now. 馃槩This PR creates the hook files in a pytest.yield_fixture and runs the test against them.