pipenv icon indicating copy to clipboard operation
pipenv copied to clipboard

`--rm` should be a command instead of an option

Open aequitas opened this issue 7 years ago • 3 comments

Currently I find it hard to (re)discover the --rm function to 'reset' an environment. For me it would make more sense to have it as a command rather than a option as it more an action taken instead of a modifier like --two/--three.

Right now it doesn't function as an modifier option as well. For example, running pipenv --rm sync would be useful as it could first remove the environment before syncing it anew, but currently skips the --rm.

My suggestion would be to move the functionality to a remove command.

aequitas avatar Oct 03 '18 11:10 aequitas

This makes sense (although making an API change at this point will be challenging)

Either way you will need to propose it as a Markdown document to the peeps folder at the root of the repository so that Kenneth sees it and can review it. Probably you can just make a document that contains what you already wrote here as an explanation. Thanks for the suggestion!

techalchemy avatar Oct 30 '18 01:10 techalchemy

I think it makes sense -- we can still support --rm for a period of time, but making it a command sounds good along with a command to activate the virtualenv only (not invoking a sub-shell).

matteius avatar Mar 04 '23 03:03 matteius

Hi @matteius, I would like to take this up. It is my first time here. Can you help me start with this?

vishalagrawal22 avatar Oct 01 '23 13:10 vishalagrawal22