Skip to content

Comments

feat(cli): add cli option to specify working path#6810

Merged
neersighted merged 4 commits intopython-poetry:masterfrom
finswimmer:feature/project-option
Nov 15, 2022
Merged

feat(cli): add cli option to specify working path#6810
neersighted merged 4 commits intopython-poetry:masterfrom
finswimmer:feature/project-option

Conversation

@finswimmer
Copy link
Member

@finswimmer finswimmer commented Oct 15, 2022 •
edited by neersighted
Loading

Pull Request Check List

Resolves: #2179, #7039

  • Added tests for changed code.
  • Updated documentation for changed code.

@finswimmer finswimmer force-pushed the feature/project-option branch 2 times, most recently from 7260721 to 0e20911 Compare October 16, 2022 06:45
@finswimmer finswimmer marked this pull request as ready for review October 16, 2022 06:46
@finswimmer finswimmer requested a review from a team October 16, 2022 06:46
@finswimmer
Copy link
Member Author

finswimmer commented Oct 16, 2022

Could need some help to write proper tests 🙏

fxtressia
fxtressia reviewed Oct 17, 2022
View reviewed changes
Copy link
Member

@fxtressia fxtressia left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM! It's pretty small sized!

Yupp, we might want some test here, but I'll assume that to be simple!

@fxtressia fxtressia added area/cli Related to the command line area/config Related to configuration management area/ux Features and improvements related to the user experience and removed area/cli Related to the command line labels Oct 17, 2022
@finswimmer finswimmer force-pushed the feature/project-option branch from 0e20911 to 4bc9a33 Compare October 30, 2022 19:21
neersighted
neersighted requested changes Oct 30, 2022
View reviewed changes
Copy link
Member

@neersighted neersighted left a comment

Choose a reason for hiding this comment

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

Some phrasing changes, and I'd like to consider a short flag ala -C.

@neersighted neersighted changed the title feat(cli): added cli option to specify path to project to run on feat(cli): add cli option to specify working path Oct 30, 2022
@finswimmer finswimmer force-pushed the feature/project-option branch from 4bc9a33 to 1680722 Compare October 31, 2022 07:11
@finswimmer finswimmer force-pushed the feature/project-option branch from 1680722 to 0375515 Compare October 31, 2022 07:27
@clintonroy
Copy link
Contributor

clintonroy commented Oct 31, 2022

-C, --directory are the common options for this in unix tools (make, tar) if you're willing to tweak the doc a tiny bit to match that.

@finswimmer finswimmer force-pushed the feature/project-option branch from 0375515 to 259cf2b Compare October 31, 2022 07:39
@finswimmer
Copy link
Member Author

finswimmer commented Nov 1, 2022

-C, --directory are the common options for this in unix tools (make, tar) if you're willing to tweak the doc a tiny bit to match that.

Fair enough. But https://xkcd.com/1168/ 🤣

neersighted
neersighted requested changes Nov 3, 2022
View reviewed changes
Copy link
Member

@neersighted neersighted left a comment

Choose a reason for hiding this comment

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

I'd like to see this as --directory, -C only, as not only does it match other tools, but --project feels more vague in intent/subject to misinterpretation.

The implementation LGTM.

@finswimmer finswimmer force-pushed the feature/project-option branch from 259cf2b to e68ea1e Compare November 3, 2022 19:22
@finswimmer finswimmer force-pushed the feature/project-option branch from e68ea1e to 34618b3 Compare November 3, 2022 19:24
@finswimmer finswimmer mentioned this pull request Nov 4, 2022
Closed
@vikram-dagger vikram-dagger mentioned this pull request Nov 8, 2022
Merged
@neersighted neersighted mentioned this pull request Nov 15, 2022
Closed
2 tasks
@neersighted neersighted linked an issue Nov 15, 2022 that may be closed by this pull request
add possibility to set the pyproject TOML from a custom path when running a command #7039
Closed
2 tasks
@neersighted neersighted mentioned this pull request Nov 15, 2022
Closed
2 tasks
@neersighted neersighted added the kind/feature Feature requests/implementations label Nov 15, 2022
@neersighted neersighted added the impact/changelog Requires a changelog entry label Nov 15, 2022
@neersighted neersighted added this to the 1.3 milestone Nov 15, 2022
@neersighted neersighted enabled auto-merge (rebase) November 15, 2022 19:07
@neersighted neersighted force-pushed the feature/project-option branch from 34618b3 to c3e48b2 Compare November 15, 2022 19:07
@neersighted neersighted merged commit 4962622 into python-poetry:master Nov 15, 2022
@radoering radoering mentioned this pull request Dec 3, 2022
Merged
@brandonchinn178 brandonchinn178 mentioned this pull request May 9, 2023
Closed
2 tasks
@github-actions
Copy link

github-actions bot commented Feb 29, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@neersighted neersighted neersighted approved these changes

+1 more reviewer

@fxtressia fxtressia fxtressia left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area/config Related to configuration management area/ux Features and improvements related to the user experience impact/changelog Requires a changelog entry kind/feature Feature requests/implementations

Projects

None yet

Milestone

1.3

4 participants

@finswimmer @clintonroy @neersighted @fxtressia