add --dry-run isn't very dry
See original GitHub issue- I am on the latest Poetry version.
- I have searched the issues of this repo and believe that this is not a duplicate.
- If an exception occurs when executing a command, I executed it again in debug mode (
-vvv
option).
- OS version and name: macOS 10.13.5
- Poetry version: Poetry 0.11.4
- pyproject.toml file: unchanged from
poetry new test
Issue
Doing poetry add --dry-run {any package}
seems to add the package to pyproject.toml
, do some work, and then revert pyproject.toml
. I noticed this because interrupting the poetry add
does not do the revert, and the dependency was effectively saved.
Issue Analytics
- State:
- Created 5 years ago
- Comments:5 (3 by maintainers)
Top Results From Across the Web
Add "dry-run" option that doesn't actually save the entities
Provide a "dry-run" option to parse and test importing the data, but not actually save it. This could work really well with #3192565:...
Read more >Instructions For Remote Dry Run ATI TEAS Exam
To access the dry run, begin by opening Google Chrome. Log into your ATI student account: My ATI tab > Test tab.
Read more >Do all git commands have a dry-run option? - Stack Overflow
Not every command would naturally support a dry run directly. git merge has its own option ( git merge --no-commit --no-ff ); but...
Read more >Dry running a function - Software Engineering Stack Exchange
The problem with this approach is that the dry run function is not maintainable at all. Any slight change in the initial function...
Read more >DRY RUN: A Memoir: MacCallum, Nikki - Amazon.com
DRY RUN : A Memoir [MacCallum, Nikki] on Amazon.com. ... Add to Cart ... her story in a way that is so incredibly...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I was working on this, I did a Draft PR #3233, I need to change to BaseException! 😃
As #3233 was merged, think this issue can be closed maybe 😃