-
-
Notifications
You must be signed in to change notification settings - Fork 678
gazelle_python.yaml will always be generated whether integrity matches or not. #2096
Copy link
Copy link
Closed
Description
🐞 bug report
Affected Rule
gazelle_python_manifest
Is this a regression?
Yes
Description
After I upgraded rules_python for a project, I found that the ci of the project failed frequently. I noticed that I was restricted by frequent requests for pypi(mirror). failed with gazelle_python_manifest.genrule. (Because I test with bazel test //...
But this is not expected, because there is integrity in my gazelle_python.yaml.
#1666 make a change that a new genrule with diff_test.
this genrule and update target are not manual, so it will be built by running bazel test //....
This obviously goes against the purpose of integrity (we don't want to regenerate gazelle_python.yaml for the same requirements).
🔬 Minimal Reproduction
🔥 Exception or Error
Any repo🌍 Your Environment
Operating System:
linux/macosOutput of bazel version:
7.2.1Rules_python version:
0.34.0Anything else relevant?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels