Remove requirements_git comments in output file #15316
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
I was reminded of this looking at #15314
Elevator pitch:
We consider
requirements_git.txt
in resolution of dependencies (so that we get dependencies of git requirements) but we remove the direct git requirements from the resultantrequirements.txt
file... but that was implemented before pip-tools changed their format. At the time it was implemented, it was fully correct because the lines it operated on looked like:But then pip-tools changed format to be like
So the prior logic would remove the
git+
lines. After the format change, it started leaving around the comment lines.I believe people have been leaving these comments lines around, mis-interpreting them as the source for the requirement above
git+foo
, which it is not.ISSUE TYPE
COMPONENT NAME