-
Notifications
You must be signed in to change notification settings - Fork 94
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
It is recommended to remove pytest-runnner from setup_requires in setup.py #168
Comments
Thanks for reporting this! What's the minimum test case for me to reproduce this? |
You could try adding
to |
Unfortunately it's not that easy: |
Turns out this wasn't too hard after all :) Fixed in master, will be released in 0.6.1 |
Due to deprecated features, it is recommended to remove
pytest-runner
fromsetup_requires
insetup.py
(source).This also causes a long trace of error when running nbstripout as a pre-commit hook. Relevant error line after running
pre-commit run -a
with the recommended yaml entry:subprocess.CalledProcessError: Command '['/Users/<user_folder>/.cache/pre-commit/repoid22b5j6/py_env-python3.9/bin/python', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/var/folders/ln/kgmmx8kn4g50tg08r0hx91800000gn/T/tmpnm00h86e', '--quiet', 'pytest-runner']' returned non-zero exit status 2.
Removing
pytest-runner
fromsetup.py
solves the issue.The text was updated successfully, but these errors were encountered: