Skip to content

Conversation

@joeybaker
Copy link
Contributor

Previously, the caches were re-initialized on every plugin initialization.
This caused an incompatibility with other tools like rebundler or
persistify.

This change keeps the original intent of creating caches when the plugin is
initialized, but ensures the process keeps the caches instead of the plugin
instance itself. It’s not really possible to have a cache conflict since
they’re keyed by file.

Fixes #32

Previously, the caches were re-initialized on every plugin
initialization. This caused an incompatibility with other tools like
rebundler or persistify.

This change keeps the original intent of creating caches when the plugin
is initialized, but ensures the process keeps the caches instead of the
plugin instance itself. It’s not really possible to have a cache
conflict since they’re keyed by file.
@joshwnj
Copy link
Member

joshwnj commented Sep 11, 2015

Great to see you got it working with rebundler!

joshwnj added a commit that referenced this pull request Sep 11, 2015
Ensure caches are persisted
@joshwnj joshwnj merged commit 880fd8c into css-modules:master Sep 11, 2015
@joeybaker joeybaker deleted the fix-32 branch September 11, 2015 21:03
@joeybaker
Copy link
Contributor Author

Thanks for the fast merge!

@mxl mxl mentioned this pull request May 17, 2017
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.

2 participants