-
-
Notifications
You must be signed in to change notification settings - Fork 51
Change default cache directory #64
Change default cache directory #64
Conversation
I am afraid it is breaking change |
@evilebottnawi what do you suggest? waiting for the next major release? |
@mistic i think yes, anyway webpack@5 have built-in cache module, so i suggest see where he stored cache and send a PR using |
@evilebottnawi I've opened the PR into the webpack repo webpack/webpack#8933 Do you think we can now merge this one here? |
@mistic no need issue 👍 We should bump major version after merge, maybe exists other issue with breaking changes? |
@evilebottnawi let me try and think a little bit! One thing I'll do is put up another PR in order to fix the build 😄 |
Let's rebase this PR to avoid CI problems |
@mistic ping me if all done for major release and we do this |
@evilebottnawi I can't think in anything more for the moment that could be a breaking change |
This PR closes #62
It is pretty simple and only change the default cache directory.
/CC @evilebottnawi