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

Switch to packaging.tags.mac_version for macOS wheel tag backfilling #7561

Merged

Conversation

chrahunt
Copy link
Member

@chrahunt chrahunt commented Jan 6, 2020

Progresses #6908. Split from #7354.

@chrahunt chrahunt added type: refactor Refactoring code skip news Does not need a NEWS file entry (eg: trivial changes) labels Jan 6, 2020
@chrahunt chrahunt marked this pull request as ready for review January 7, 2020 01:09
src/pip/_internal/pep425tags.py Outdated Show resolved Hide resolved
Makes the next change easier to review.
This function takes care of the version iteration AND architecture
determination internally.
@chrahunt chrahunt force-pushed the maint/use-packaging-tags-mac-versions branch from f3a220e to bc20a98 Compare January 7, 2020 07:17
@pradyunsg pradyunsg removed the skip news Does not need a NEWS file entry (eg: trivial changes) label Jan 7, 2020
@pradyunsg
Copy link
Member

(dropped the trivial label, based on @xavfernandez's review comment)

@xavfernandez xavfernandez added the skip news Does not need a NEWS file entry (eg: trivial changes) label Jan 7, 2020
@xavfernandez xavfernandez merged commit 9c64c82 into pypa:master Jan 7, 2020
@chrahunt chrahunt deleted the maint/use-packaging-tags-mac-versions branch January 7, 2020 21:59
@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Feb 6, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation skip news Does not need a NEWS file entry (eg: trivial changes) type: refactor Refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants