Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

deep namespaces: cache invalidation #200

Closed
benmosher opened this issue Mar 2, 2016 · 0 comments · Fixed by #212
Closed

deep namespaces: cache invalidation #200

benmosher opened this issue Mar 2, 2016 · 0 comments · Fixed by #212
Labels
bug
Milestone
v1.0.4

Comments

@benmosher
Copy link
Member

need to convert deep namespaces to export map references, for cache correctness. if the internal namespaces are only referenced indirectly, there is no opportunity to reparse on mtime change.

impacts:

  • eslint_d
  • eslint-loader
  • fs cache (FS cache #199 -- would be simpler to serialize without duplication)
@benmosher benmosher added the bug label Mar 2, 2016
@benmosher benmosher added this to the patch milestone Mar 2, 2016
@benmosher benmosher mentioned this issue Mar 2, 2016
Closed
benmosher added a commit that referenced this issue Mar 4, 2016
@benmosher
deep caching first draft (#200). lots of breaking changes still
a13d5c0
@benmosher benmosher mentioned this issue Mar 6, 2016
Closed
benmosher added a commit that referenced this issue Mar 11, 2016
@benmosher
deep caching first draft (#200). lots of breaking changes still
4999312
@benmosher benmosher mentioned this issue Mar 11, 2016
Merged
This was referenced Apr 15, 2018
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Milestone
v1.0.4
Development

Successfully merging a pull request may close this issue.

deep namespaces: correct caching import-js/eslint-plugin-import
1 participant
@benmosher