-
Notifications
You must be signed in to change notification settings - Fork 30k
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
module: simplify findPackageJSON
implementation
#55543
Conversation
Review requested:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #55543 +/- ##
==========================================
- Coverage 88.43% 88.32% -0.12%
==========================================
Files 654 654
Lines 187657 187629 -28
Branches 36117 36041 -76
==========================================
- Hits 165951 165718 -233
- Misses 14947 15137 +190
- Partials 6759 6774 +15
|
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.
Thanks! I was gonna do whilst my friend was getting ready this morning ;)
Could you also restore #55412 (review)
I dunno how it got removed.
|
Landed in c0db893 |
PR-URL: nodejs#55543 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
PR-URL: #55543 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
PR-URL: nodejs#55543 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
This commit does not land cleanly on |
How do I find the commit hash I should use to create the backport? Is it the one in Also, can I combine this backport with the one for #55412, since this is directly dependent/related to that? |
You can use the one given on #55543 (comment), or the one that landed on v23.x. It should definitely be combined yes |
PR-URL: nodejs#55543 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
PR-URL: nodejs#55543 Backport-PR-URL: nodejs#56074 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> PR-URL: nodejs#55412
Fixes: #55412 (comment)