Take vitest 4.1.11 and cut 2.7.6 - #76
Merged
Merged
Conversation
Workspace dependency sweep. vitest and @vitest/coverage-v8 move 4.1.10 to 4.1.11; both are devDependencies, so nothing reaches a consumer. Version bumped to 2.7.6 so the SDK ships as part of a coordinated release across the SDKs, commerce libraries and pay, rather than because this change requires callers to act. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.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.
Summary
Workspace dependency sweep for this repo.
vitestand@vitest/coverage-v8move4.1.10 to 4.1.11. Both are devDependencies, so nothing here reaches a consumer.
The version bump to 2.7.6 is deliberate rather than required: nothing in this diff
obliges a caller to act. It exists so the SDK ships as tier 1 of a coordinated
release across the SDKs, the commerce libraries and pay, which was Varun's call
after I flagged that by the usual "bump only where a consumer must act" rule this
repo did not need a release at all.
Held deliberately, with registry entries already covering them: eslint 10 and
TypeScript 7 both stay on the 9.x and 6.x lines.
Worked with: Varun.
Type of change
Public API
None. No exported type, function signature, wire format or response shape changes
in this diff. The only non-lockfile edit is the
versionfield.Test plan
Ran the repo's full gate set locally on this branch:
bun run typecheckcleanbun run lintclean (eslint, max-warnings 0)bun run test: 7 files passed, 1 skipped; 166 tests passed, 9 skippedosv-scanneroverbun.lock: no issues found across 504 packagesNo tests were added or changed, because no behavior changed.
Checklist
(no new behavior; the existing suite passes unchanged on 4.1.11)
(public surface did not change, so nothing to update)
Deliberately not done
^2.7.6anywhere. That version does not existon npm yet, so raising a floor now would make
bun installunresolvable in everyconsumer. The existing ranges already accept it, so no consumer edit is needed;
pinning to the exact tested combination is a follow-up after this publishes.
v*, and that is permanent, so thetag is left for a deliberate step after this merges.