feat: cacheKeys option
#320
Conversation
| @@ -44,6 +44,7 @@ module.exports = { | |||
| |**`include`**|`{RegExp\|Array<RegExp>}`|`undefined`|Files to `include`| | |||
| |**`exclude`**|`{RegExp\|Array<RegExp>}`|`undefined`|Files to `exclude`| | |||
| |**`cache`**|`{Boolean\|String}`|`false`|Enable file caching| | |||
| |**`cacheKeys`**|`{Function(defaultCacheKeys, file) -> {Object}}`|`defaultCacheKeys => defaultCacheKeys`|Allows you to override default cache keys| | |||
There was a problem hiding this comment.
What about the 3rd param?
There was a problem hiding this comment.
Don't understand 😄
There was a problem hiding this comment.
@evilebottnawi sorry. that means "I agree, and have the same comment"
There was a problem hiding this comment.
@shellscape i don't understand what i will do 😄 3rd param? What is mean?
There was a problem hiding this comment.
Example below:
[
new UglifyJsPlugin({
cache: true,
cacheKeys: (defaultCacheKeys, file, options) => {
// ^^^^^^^
defaultCacheKeys.myCacheKey = 'myCacheKeyValue';
return defaultCacheKeys;
},
})
]I now understand that my comment was wrong. The signature is correct, the example is not.
alexander-akait
force-pushed
the
feat-custom-cache-keys
branch
from
3e9acfa
to
142497e
Compare
| @@ -44,6 +44,7 @@ module.exports = { | |||
| |**`include`**|`{RegExp\|Array<RegExp>}`|`undefined`|Files to `include`| | |||
| |**`exclude`**|`{RegExp\|Array<RegExp>}`|`undefined`|Files to `exclude`| | |||
| |**`cache`**|`{Boolean\|String}`|`false`|Enable file caching| | |||
| |**`cacheKeys`**|`{Function(defaultCacheKeys, file) -> {Object}}`|`defaultCacheKeys => defaultCacheKeys`|Allows you to override default cache keys| | |||
There was a problem hiding this comment.
Example below:
[
new UglifyJsPlugin({
cache: true,
cacheKeys: (defaultCacheKeys, file, options) => {
// ^^^^^^^
defaultCacheKeys.myCacheKey = 'myCacheKeyValue';
return defaultCacheKeys;
},
})
]I now understand that my comment was wrong. The signature is correct, the example is not.
README.md
Outdated
| [ | ||
| new UglifyJsPlugin({ | ||
| cache: true, | ||
| cacheKeys: (defaultCacheKeys, file, options) => { |
There was a problem hiding this comment.
Remove the 3rd argument please
There was a problem hiding this comment.
/cc @ooflorent done, need approve 😄
alexander-akait
force-pushed
the
feat-custom-cache-keys
branch
from
142497e
to
2370e47
Compare
No description provided.