-
Notifications
You must be signed in to change notification settings - Fork 3
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
update webpack plugin #66
Conversation
3f4f8f5
to
de573d7
Compare
1a3d9d4
to
87875b4
Compare
BREAKING CHANGE: requires webpack >=5 fixes #58
87875b4
to
46b8b3a
Compare
@@ -7,6 +7,7 @@ | |||
"functions": 90, | |||
"statements": 90, | |||
"exclude": [ | |||
"test/**" | |||
"test/**", | |||
"src/app/utils/resolveDirectory.ts" |
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.
Why do we need that?
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.
This file is injected into the bundle, and when using snapshot testing on CI (with coverage enabled), it injects instrumented code, thus making inconsistent matching on local and CI tests.
No description provided.