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

Rename {setup_py_dir -> unpacked_source_directory} #7086

Merged
merged 1 commit into from
Sep 27, 2019

Conversation

pradyunsg
Copy link
Member

Why: Because PEP 517 means you won't necessarily have a setup.py in your unpacked sources.

Why: Because PEP 517 means you won't have a setup.py in your unpacked sources.
@pradyunsg pradyunsg added type: refactor Refactoring code skip news Does not need a NEWS file entry (eg: trivial changes) labels Sep 26, 2019
@chrahunt
Copy link
Member

Is there a name that would be closer to "the directory which has the setup.py or pyproject.toml file for a project"? As it is it may be confused with "the root of the thing that was unpacked or downloaded", which wouldn't be the case for VCS requirements with subdirectories.

@pradyunsg
Copy link
Member Author

Is there a name that would be closer to "the directory which has the setup.py or pyproject.toml file for a project"?

Naming things is difficult. :(

I couldn't come up with anything better tbh. I do think this, as is, is an improvement over status quo so there's that. If someone has a suggestion that we have consensus on, this name is now unique within the codebase, so changing it is a simple find-replace for me.

@pradyunsg pradyunsg merged commit 33b0240 into pypa:master Sep 27, 2019
@pradyunsg pradyunsg deleted the rename-setup_py_dir branch September 27, 2019 04:59
@pradyunsg
Copy link
Member Author

Hurrah! Thanks for taking a look @chrahunt! ^>^

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

2 participants