Repository navigation
chore(deps): bump four npm packages in one branch - #66
Merged
Merged
Conversation
Dependabot opened #64 (production, 1) and #65 (development, 3) against the same base. Merging them one at a time would make the other wait for a rebase and burn a second CI round, so consolidate both into a single green tree. - Production: lucide-react 1.46.0 -> 1.47.0 - Development: @types/node 26.6.1 -> 26.6.2, jsdom 30.0.1 -> 30.1.1, oxlint 1.83.0 -> 1.85.0 - No direct majors; the rest of the lock diff is transitive under jsdom and the oxlint platform bindings, and symbol-tree drops out because jsdom 30.1 no longer uses it - The branches merged without conflict, so the lock is the exact union of the #64 and #65 locks with no regeneration and no drift - Gates on the merged tree: lint clean (same 7 warnings as main), build byte-identical to main, 199 tests pass across 4 runs Closes #64 Closes #65 Tags: #deps #dependabot Co-Authored-By: htjulia <htjulia1@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dependabot opened two PRs against the same base commit (
834cbf7): #64 (production, 1 package) and #65 (development, 3 packages). Merging them one at a time would make the other wait for a rebase and burn a second CI round, so this consolidates both into a single green tree.What changed
Production (from #64)
lucide-reactDevelopment (from #65)
@types/nodejsdomoxlintThe rest of the lock diff is transitive under
jsdom(@asamuzakjp/css-color,@asamuzakjp/dom-selector,@csstools/*,bidi-js,html-encoding-sniffer,undici,w3c-xmlserializer,whatwg-url) and the@oxlint/binding-*platform packages.symbol-treedrops out becausejsdom30.1 no longer depends on it; nothing in this repo imports it. No direct majors — every direct bump stays inside its existing semver range.Lock integrity
The two branches merged without conflict, so the lock was not regenerated: it is exactly the union of the #64 and #65 locks.
lucide-reactis the 1.47.0 Dependabot verified; there is no drift to pin back.Verification on the merged tree
npm run lint: clean. The 7only-export-componentswarnings are the same ones already onmain, sooxlint1.85 adds none.npm run build: ok. The output bundle is byte-identical tomain(index-v-UcBT-i.js, 520.11 kB / gzip 164.32 kB), because the new icons are unused and tree-shaken.npm test: 17 files / 199 tests pass, run 4 times in a row with no failures (the knownquota-pressureflake did not show up).jsdom30.1 adds aQuotaExceededErrorclass.use-trace-store.tsclassifies errors withinstanceof DOMException, which the new class still satisfies. The storage-failure, retention-limit and quota-pressure suites all pass.Not caused by this PR
npm auditreports 6 advisories (4 moderate, 2 high:fast-uri,hono,ip-address,js-yaml,qs,undici).mainat834cbf7reports the same 6, all dev-only. This PR actually narrows one:jsdom's nestedundicimoves 8.10.0 -> 8.11.0, out of the 8.1.0 - 8.10.1 range. The remainingundicihit is a different copy on 7.x. Tracked separately.Closes #64
Closes #65