Repository navigation
Revert "ci(deps): Dedupe lockfile on Dependabot PRs (#25911)" - #25962
Merged
Merged
Conversation
Contributor
size-limit report 📦Compared: 76b5bda (PR test merge using develop at a3dae70) Against: a3dae70 (develop) · base run #64482 ✅ Baseline matches: this diff contains only this PR's changes.
|
Contributor
Optional E2E testsSome optional checks were not successful. They do not block the merge. 2 not successful, 52 successful. See the run.
Successful checks
|
chargome
approved these changes
Oct 30, 2026
chargome
marked this pull request as ready for review
October 30, 2026 18:11
chargome
requested review from
andreiborza,
logaretm,
msonnb and
mydea
and removed request for
a team
October 30, 2026 18:11
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.
Why: With
autoDedupeenabled in #25961, Dependabot now writes an already-deduped lockfile, so the dedupe workflow from #25911 has nothing left to do. After #25961 merged, Dependabot rebased #25955, #25956 and #25957. On all three,pnpm dedupemade no changes: the upload step was skipped, no artifact was created and no dedupe commit was pushed.Check lockfilepassed on #25955 and #25957. #25956 failspnpm installwithERR_PNPM_IGNORED_BUILDS, which is a separate problem.How to review: This is a clean
git revertof bc8fdf1. It also restores the optional E2E condition inbuild.ymlto thegithub.actorcheck, since the author-based check was only needed once the dedupe app started pushing to Dependabot branches.via Charly Gomez.
--
View Session [Sentry]