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

feat: remove all relevant directories on uninstall #831

Merged
merged 1 commit into from Feb 24, 2022

Conversation

mdonnalley
Copy link
Contributor

@mdonnalley mdonnalley commented Feb 17, 2022

Remove the following on uninstall:

  • config.dataDir
  • config.cacheDir
  • config.configDir

for macos only

@W-10695889@

@mdonnalley mdonnalley self-assigned this Feb 17, 2022
peternhale
peternhale previously approved these changes Feb 17, 2022
peternhale
peternhale previously approved these changes Feb 17, 2022
@mdonnalley mdonnalley force-pushed the mdonnalley/update-uninstall-scripts branch from b2a5ba0 to 91c1c0e Compare February 17, 2022 17:32
@RodEsp RodEsp merged commit acd986d into main Feb 24, 2022
@RodEsp RodEsp deleted the mdonnalley/update-uninstall-scripts branch February 24, 2022 17:48
gaelmotte pushed a commit to gaelmotte/oclif that referenced this pull request Oct 24, 2022
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

Successfully merging this pull request may close these issues.

None yet

3 participants