Skip to content

Comments

fix(cli): make config relocation instructions more explicit#7140

Merged
Secrus merged 7 commits intopython-poetry:masterfrom
cpate4:GH-6864_make-config-file-instructions-more-explicit
Jan 22, 2023
Merged

fix(cli): make config relocation instructions more explicit#7140
Secrus merged 7 commits intopython-poetry:masterfrom
cpate4:GH-6864_make-config-file-instructions-more-explicit

Conversation

@cpate4
Copy link
Contributor

@cpate4 cpate4 commented Dec 4, 2022

Fixes #6864

Pull Request Check List

Resolves: #6864

With regard to the boxes below, I am not sure if there are tests to write for this as the fix is pertains to "copy" for the deprecation message (as opposed to logic). Happy to update any documentation the team feels is relevant.

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

@cpate4
Copy link
Contributor Author

cpate4 commented Dec 20, 2022

pre-commit.ci autofix

Co-authored-by: Bartosz Sokorski <[email protected]>
@cpate4 cpate4 requested a review from Secrus December 20, 2022 21:10
@cpate4
Copy link
Contributor Author

cpate4 commented Dec 20, 2022

Thank you, @Secrus, for the feedback!

@Secrus
Copy link
Member

Secrus commented Jan 22, 2023

@cpate4 could you please resolve the conflict?

@Secrus
Copy link
Member

Secrus commented Jan 22, 2023

pre-commit.ci autofix

@Secrus Secrus merged commit e1504ed into python-poetry:master Jan 22, 2023
@cpate4
Copy link
Contributor Author

cpate4 commented Jan 23, 2023

@Secrus - Thank you! This is my first (ever) open source contribution. 🙂

@cpate4 cpate4 deleted the GH-6864_make-config-file-instructions-more-explicit branch January 23, 2023 11:20
@github-actions
Copy link

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.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make config file relocation instructions more explicit

2 participants