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

Enhancement: CLI command to clear the Thaw cache #64

Merged
merged 2 commits into from
Oct 21, 2020

Conversation

badalsarkar
Copy link
Contributor

Fixes #62 .
A new sub-command clean has been added. Running thaw clean will empty the root cache folder.

I have used apache commons io. Let me know if that is alright. Also, let me know if the handling of IOException is okay or you want me to change something.

I have also updated the THIRD_PARTY_NOTICE.md file to include apache commons io license.

badalsarkar and others added 2 commits October 21, 2020 09:50
Implements CLI command to clear the Thaw cache.
Copy link
Owner

@bennyboer bennyboer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Just added some attribution notices for commons IO 😄

@bennyboer bennyboer merged commit 85f4c7c into bennyboer:master Oct 21, 2020
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.

CLI command to clear the Thaw cache
2 participants