Skip to content
This repository was archived by the owner on Oct 27, 2020. It is now read-only.

feat: use find-cache-dir to determinate the default directory (options.cacheDirectory) #21

Merged
merged 1 commit into from
Nov 30, 2017

Conversation

alexander-akait
Copy link
Member

Many good project use find-cache-dir for standardize the cache storage path. Also no temp cache files into root project (no need add pattern to .gitignore).

@alexander-akait alexander-akait force-pushed the feature-use-find-cache-dir branch from b5938d3 to bec9bd3 Compare November 14, 2017 11:02
@michael-ciniawsky michael-ciniawsky changed the title feature: use find-cache-dir for determinate cache path feat: use find-cache-dir to determinate the default path (options.cacheDirectory) Nov 14, 2017
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

I much in favor of this 👍

@michael-ciniawsky michael-ciniawsky added this to the 1.1.0 milestone Nov 14, 2017
@michael-ciniawsky michael-ciniawsky changed the title feat: use find-cache-dir to determinate the default path (options.cacheDirectory) feat: use find-cache-dir to determinate the default directory (options.cacheDirectory) Nov 14, 2017
Copy link
Member

@sokra sokra left a comment

Choose a reason for hiding this comment

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

looks good to me.

As this is changing the default it should be a major version.

@michael-ciniawsky
Copy link
Member

michael-ciniawsky commented Nov 15, 2017

@evilebottnawi Could you rebase please ? :)

@alexander-akait alexander-akait force-pushed the feature-use-find-cache-dir branch from bec9bd3 to 55c59b8 Compare November 17, 2017 09:43
@alexander-akait
Copy link
Member Author

@michael-ciniawsky done 👍

@joshwiens joshwiens merged commit 58dc5ae into next Nov 30, 2017
@joshwiens joshwiens deleted the feature-use-find-cache-dir branch November 30, 2017 17:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants