search

lasso-js / lasso

Advanced JavaScript module bundler, asset pipeline and optimizer
582 stars 75 forks source link

Move setting the cache of the lassoContext to be last #273

Closed scttdavs closed 6 years ago

scttdavs commented 6 years ago

Two events are fired after the cache is set on the lassoContext:

The payload of lassoCacheCreated actually includes lassoContext.options. But since options is set after the cache is set, it will always be undefined.

buildCacheKey also includes the lassoContext in its event payload, so this would allow options to also be available there.

austinkelleher commented 6 years ago

Will merge and publish when the CI passes

austinkelleher commented 6 years ago

Published lasso@3.2.1. Thanks again @scttdavs!