-
-
Notifications
You must be signed in to change notification settings - Fork 151
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
Strange lint CI failure? #545
Comments
I could only skim through them, but the failure may be related to these issues:
Would updating setuptools in the runner's Python installation help?
I'm assuming here that |
Yep! I tried Very strange I can't replicate it locally either with makes me think it's a GHA runner related thing? |
You need to invoke this outside of the virtualenv created by Nox. Alternatively, use the full path:
|
Sorry I didn't see you were already doing this. |
There was a new release of virtualenv today which bumped the embedded version of setuptools, that's next on my list! |
It looks like virtualenv is the culprit! See my now passing lint run here Pinning virtualenv to 20.10.0 inside the lint session has fixed it. pre-commit depends on So it must have pulled in today's virtualenv 20.11.0 bumps the embedded setuptools from 58.3.0 to 59.4.0 so I guess there is a breaking change in there somewhere. Weird that I couldn't reproduce locally though, even with these versions |
Describe the bug
Noticed a lint job failure after merging #543 (which had no such failure on the PR):
See here
I tried re-running but got the same outcome, then on my fork I tried adding
python -m pip install --upgrade pip setuptools wheel
to all CI jobs but got the same outcomeEdit: Also tried injecting setuptools directly into the nox session with
session.install("setuptools", "pre-commit")
, same resultEdit2: It seemed to always happen on
isort
so I tried removing it from the pre-commit config but it then just failed onblack
with the exact same errorHow to reproduce
Look at the CI lint job, it seems to happen on re-runs
Expected behavior
Should run to success like in the PR
The text was updated successfully, but these errors were encountered: