Skip to content

Uninstall script deletes config#7

Merged
lordcodes merged 1 commit intomasterfrom
uninstall-delete-config
Mar 11, 2020
Merged

Uninstall script deletes config#7
lordcodes merged 1 commit intomasterfrom
uninstall-delete-config

Conversation

@lordcodes
Copy link
Owner

Checklist

  • I've read the guide for contributing.
  • I've checked there are no other open pull requests for the same change.
  • I've made sure all the tests pass with swift test.
  • I've formatted all code changes with swift run task format.
  • I've ran all checks with swift run task lint.
  • I've updated documentation if needed.
  • I've added or updated tests for changes.

Reason for change

The uninstall task left the imagecram config directory containing the TinyPNG API key behind.

Description

Simply make the task delete the config directory.

Closes #3

@lordcodes lordcodes self-assigned this Mar 11, 2020
@lordcodes lordcodes added this to the v1.0 milestone Mar 11, 2020
@lordcodes lordcodes merged commit 4263447 into master Mar 11, 2020
@lordcodes lordcodes deleted the uninstall-delete-config branch March 11, 2020 20:43
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.

Uninstall task should remove config

1 participant