-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
deps: update cjs-module-lexer to 1.4.0 #54713
Conversation
Review requested:
|
The issue turned out to be the Windows line endings here. Apparently I'm the only one that develops on Windows! Pushed up a commit that should resolve the problem. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54713 +/- ##
=======================================
Coverage 87.61% 87.61%
=======================================
Files 650 650
Lines 182835 182829 -6
Branches 35382 35386 +4
=======================================
- Hits 160185 160181 -4
- Misses 15925 15930 +5
+ Partials 6725 6718 -7 |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
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 once CI is green
Landed in 01bf4a1 |
PR-URL: #54713 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This is an automated update of cjs-module-lexer to 1.4.0.