Skip to content

Clean up trivial directory option handling #7467

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

Merged
merged 4 commits into from
Dec 12, 2019
Merged

Conversation

chrahunt
Copy link
Member

Some minor cleanup before further refactoring in this area.

This will let us reduce duplication of this option normalization in a
few of our commands.
Technically this changes behavior in DownloadCommand, which does not
make build_dir absolute, but given that it is used the same way as in
InstallCommand and WheelCommand (which do make it absolute), it should
not have any visible impact.
@chrahunt chrahunt added skip news Does not need a NEWS file entry (eg: trivial changes) refactor type: refactor Refactoring code and removed refactor labels Dec 12, 2019
@chrahunt chrahunt marked this pull request as ready for review December 12, 2019 05:09
@chrahunt chrahunt merged commit fe320a3 into master Dec 12, 2019
@chrahunt chrahunt deleted the refactor/cleanup-cli-opts branch December 12, 2019 13:46
@chrahunt
Copy link
Member Author

Thanks for reviewing!

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jan 11, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Jan 11, 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