lirantal/nodejs-cli-apps-best-practices

Cleanup configuration files (if any) when uninstalling the package

m-sureshraj opened this issue · 5 comments

m-sureshraj commented

This could be a new topic. The CLI application should remove it's internal configuration files (if there are any) when uninstalling the package. We can use NPM's pre or post uninstall script for this purpose.

https://docs.npmjs.com/misc/scripts

lirantal commented

Sounds like a good idea to add in the Distribution section 👍

  • 👍1
  • 🎉1
lirantal commented

@m-sureshraj perhaps wasn't clear from my previous comment - you are more than welcome to PR this idea and we can work out the exact semantics and suggestions via the PR.

  • 👍1
m-sureshraj commented

Sure @lirantal. I will create a PR for this topic. Thanks.

jasonkarns commented

Even better if the user can opt-in to preserving config files. Many times it's valuable for the config to persist across installations.

  • 👍1
m-sureshraj commented

Hi, I've created PR for this topic. Please review - #16

cc: @lirantal @jasonkarns