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

Create cache directory path if it does not exist #174

Merged
merged 3 commits into from
Sep 12, 2021

Conversation

SimplyDanny
Copy link
Contributor

This PR tries to resolve #167. If TEALDEER_CACHE_DIR is set but does not exist, the complete path is just created as suggested in the issue. The user will be informed about the creation.

@dbrgn dbrgn added this to the v1.5.0 milestone Apr 14, 2021
Copy link
Collaborator

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

Thank you for contributing 🚀! Everything looks correct, I only have some suggestions regarding code structure and naming. That said, this is of cource open for discussion :)

src/cache.rs Outdated Show resolved Hide resolved
src/cache.rs Outdated Show resolved Hide resolved
tests/lib.rs Outdated Show resolved Hide resolved
tests/lib.rs Outdated Show resolved Hide resolved
@niklasmohrin
Copy link
Collaborator

@dbrgn Any chance you can take a quick look at #174 (comment) so this can move forward? :^)

@dbrgn dbrgn added the ready-for-review The PR can be reviewed and is waiting for the maintainers. label Sep 7, 2021
@dbrgn dbrgn changed the title Fix #167: Create cache directory path if it does not exist Create cache directory path if it does not exist Sep 7, 2021
@dbrgn dbrgn added waiting-for-author The PR needs an update before it can be considered for merging. and removed ready-for-review The PR can be reviewed and is waiting for the maintainers. labels Sep 7, 2021
Copy link
Collaborator

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

only two nitpicks left from me 👍

src/cache.rs Outdated Show resolved Hide resolved
src/cache.rs Outdated Show resolved Hide resolved
@niklasmohrin niklasmohrin added ready-for-review The PR can be reviewed and is waiting for the maintainers. and removed waiting-for-author The PR needs an update before it can be considered for merging. labels Sep 12, 2021
Copy link
Collaborator

@dbrgn dbrgn 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 now, thank you @SimplyDanny (and @niklasmohrin for the reviews)!

@dbrgn dbrgn merged commit 1471b1d into tealdeer-rs:master Sep 12, 2021
@SimplyDanny SimplyDanny deleted the create-cache-directory branch September 13, 2021 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-review The PR can be reviewed and is waiting for the maintainers.
Development

Successfully merging this pull request may close these issues.

tldr --clear-cache with $TEALDEER_CACHE_DIR requires manual mkdir to rebuild cache
3 participants