chore(deps): trivial major-version dependency migrations - #1036
Open
ByronDWall wants to merge 2 commits into
Open
chore(deps): trivial major-version dependency migrations#1036ByronDWall wants to merge 2 commits into
ByronDWall wants to merge 2 commits into
Conversation
|
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
Bumps
find-upfrom7.0.0to8.0.0(devDependency only).Included
7.0.0→8.0.0(devDependency). A full-repo grep confirmed the only occurrences offind-upare thepackage.jsonversion string itself and a mention in.github/renovate.json— there are no source-code imports offind-upanywhere in this repo, and nothing importspathExistsfrom it. This bump required no call-site code changes, purely a manifest + lockfile update..nvmrc.path-exists— consumers must importpath-existsdirectly if they used that re-export. Not used in this repo.Excluded as not trivial (context for reviewers)
The Dependency Dashboard has several other pending major-version bumps. Each was evaluated and excluded from this PR because it's a framework/tooling migration or otherwise out of scope for a "trivial mechanical bump" PR, not because it's risky in a specific way that was investigated in depth here:
actions/checkout,actions/setup-node,pnpm/action-setup(GH Actions majors)@commercetools-frontend/*application-kit packages@commitlint/cli,@commitlint/config-conventional,lint-staged(git-workflow-tools major)@graphql-codegen/*packageseslint,eslint-formatter-pretty@commercetools/platform-sdk(v9)Flagged for manual review (not blind-included)
@commitlint/cli,@commitlint/config-conventional, andlint-stagedwere specifically evaluated for this batch and not included. They are git-hook/workflow tooling spanning two majors each, with real config-schema risk (commitlint rule changes, lint-staged config format changes) rather than a purely mechanical version bump. Recommending these go through manual review rather than blind inclusion in a "trivial" batch.Changeset
No changeset was added. Per
CONTRIBUTING.md, "a changeset is not required, as things like documentation or other changes in the repository itself generally don't need a changeset," and PR #1033 (a comparable pure dependency-bump/consolidation PR) included zero changeset files. This PR mirrors that convention: it's a devDependency-only bump with no consumer-facing package version change.Verification
Ran the full local verification suite in an isolated worktree:
pnpm typecheck(tsc --noEmit) — passpnpm build(preconstruct build) — passpnpm test(jest) — 535 suites / 1691 tests — passpnpm lint(jest --config jest.eslint.config.js) — 2820 suites — passAll green, consistent with
find-uphaving zero source-code call sites in this repo.Notes
chore/renovate-batch-consolidation), which was left untouched.