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

remove stale dependencies #15

Open
Alloyed opened this issue Sep 6, 2017 · 1 comment
Open

remove stale dependencies #15

Alloyed opened this issue Sep 6, 2017 · 1 comment

Comments

@Alloyed
Copy link
Owner

Alloyed commented Sep 6, 2017

this would either be a flag to purge, or a custom command (clean?).
it might make sense to remove purge, or replace it with rm -rf rocks/.
although having it adds a sense of discoverability, you can read the help and know how to empty a tree

@pablomayobre
Copy link
Contributor

My suggestion is the following:

  • clean should be a command that gives you a clean state (currently this command is called purge)
  • prune should be a command that cleans the unneeded/stale dependencies (this is the behaviour of npm prune)
  • deps --upgrade should be a command to install new dependencies and remove unneeded/stale ones (this is not actually needed but could be a nice upgrade)

Note that I have removed purge in favour of clean, and added prune which has the described behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants