Skip to content
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

Add missing verbosity flag #875

Merged
merged 6 commits into from
Feb 5, 2022
Merged

Add missing verbosity flag #875

merged 6 commits into from
Feb 5, 2022

Conversation

q0w
Copy link
Contributor

@q0w q0w commented Jan 31, 2022

#874

Pull Request Check List

  • A news fragment is added in news/ describing what is new.
  • Test cases added for changed code.

Describe what you have changed in this PR.

@CharString
Copy link

This would solve #874

@pawamoy
Copy link
Contributor

pawamoy commented Jan 31, 2022

Is the flag backported to previous pip versions? If not, and if always using it in PDM, then I think the minimum required pip version in pyproject.toml should also be upgraded? unpack_url does not swallow extra keyword arguments.

@frostming
Copy link
Collaborator

You need to ensure the fix works for the specified version range of pip in dependencies. But before that, I will cut a new release pinning pip<22 to fix the breakage.

Thanks

@q0w
Copy link
Contributor Author

q0w commented Jan 31, 2022

Should I check if unpack_url has verbosity flag in function signature?

@pawamoy
Copy link
Contributor

pawamoy commented Jan 31, 2022

This, or check pip's version using importlib.metadata.

pdm/models/candidates.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@frostming frostming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be done in the pip_shims module, and other modules just import and use it. That is what the name pip_shims stands for.

@frostming frostming linked an issue Feb 3, 2022 that may be closed by this pull request
@q0w
Copy link
Contributor Author

q0w commented Feb 3, 2022

This needs to be done in the pip_shims module, and other modules just import and use it. That is what the name pip_shims stands for.

Ok. Should it be tested somehow?

pdm/models/pip_shims.py Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Feb 5, 2022

Codecov Report

Merging #875 (b78b7f1) into main (40cd92b) will decrease coverage by 0.51%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #875      +/-   ##
==========================================
- Coverage   83.86%   83.34%   -0.52%     
==========================================
  Files          74       74              
  Lines        6353     6359       +6     
  Branches     1315     1316       +1     
==========================================
- Hits         5328     5300      -28     
- Misses        707      746      +39     
+ Partials      318      313       -5     
Flag Coverage Δ
unittests 83.11% <33.33%> (-0.52%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pdm/models/pip_shims.py 86.79% <33.33%> (-3.21%) ⬇️
pdm/models/setup.py 42.29% <0.00%> (-10.28%) ⬇️
pdm/core.py 87.06% <0.00%> (-1.43%) ⬇️
pdm/models/repositories.py 76.44% <0.00%> (-0.89%) ⬇️
pdm/formats/poetry.py 91.79% <0.00%> (-0.75%) ⬇️
pdm/installers/installers.py 83.92% <0.00%> (-0.60%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40cd92b...b78b7f1. Read the comment docs.

@frostming frostming merged commit 42edab5 into pdm-project:main Feb 5, 2022
@frostming
Copy link
Collaborator

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pip 22.0.1 broke pip_shims.unpack_url
5 participants