Repository navigation
fix: update binary-split repository - #1142
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: ce1a4b02-26ae-4c30-a086-491e98d1e6da
|
This PR should have a commit message |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1142 +/- ##
=======================================
Coverage 96.20% 96.20%
=======================================
Files 29 29
Lines 2213 2213
=======================================
Hits 2129 2129
Misses 84 84 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
I didn't notice this in time to add to this PR, however one of the maintainers ( @mourner is stil reachable, however there haven't been any repo updates since Dec 2023. The package was last published to https://www.npmjs.com/package/binary-split as 1.0.5 on Oct 23, 2018. Edit: see #1166 |
|
It also fails in Windows on all Node.js versions. Currently it is marked as flaky for Windows. It would be better to skip it for Windows instead. I'll follow up with a separate PR for this. Edit: see #1167 |
Updates the
binary-splitlookup entry to use the currentmax-mapperrepository owner.The package metadata published for
binary-split@1.0.5still points tomaxogden/binary-split. npm/cli uses the CITGM lookup repository override when generating its integration matrix, preventing the stale codeload URL from returning a 404.Addresses npm/cli#10020.
Validation:
jq empty lib/lookup.jsonandgit diff --check. The full test suite was not run.