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.
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
fix module cache clearing in live mode for Node 12 (closes #4052) #4090
Conversation
❌ Tests for the commit a6d615d have failed. See details: |
❌ Tests for the commit a6d615d have failed. See details: |
❌ Tests for the commit 982d567 have failed. See details: |
❌ Tests for the commit 982d567 have failed. See details: |
✅ Tests for the commit 982d567 have passed. See details: |
@@ -57,7 +57,7 @@ export default class ModulesGraph { | |||
if (!cache[node]) | |||
return; | |||
|
|||
cache[node] = null; | |||
cache[node] = void 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think if we can just delete cache[node]
, can't we?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can
❌ Tests for the commit 87f6ea2 have failed. See details: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, don't merge until testcafe@1.4.0
is released.
✅ Tests for the commit 87f6ea2 have passed. See details: |
the cause is here https://github.com/nodejs/node/blob/master/lib/internal/modules/cjs/loader.js#L617
and the pull is here nodejs/node@0f190a5