Skip to content

chore: switch renovate lock file maintenance to a reviewed PR#1978

Closed
vdusek wants to merge 1 commit into
masterfrom
chore/renovate-lock-maintenance-release-age
Closed

chore: switch renovate lock file maintenance to a reviewed PR#1978
vdusek wants to merge 1 commit into
masterfrom
chore/renovate-lock-maintenance-release-age

Conversation

@vdusek

@vdusek vdusek commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Lock file maintenance was effectively stuck. Renovate cannot yet pass an age constraint to uv lock (renovatebot/renovate#41654), so minimumReleaseAge only added a renovate/stability-days check that blocked the merge while uv lock --upgrade still resolved to the newest versions. Combined with branch automerge and strict internal checks, the branch was rebased and regenerated on every master change, so the check never cleared and the lockfile refresh never landed, leaving transitive dependencies frozen.

This drops automerge/automergeType and sets minimumReleaseAge: "0 days" on lockFileMaintenance, so the refresh opens a regular PR for review instead: the PR reliably appears (no stuck stability check) and the transitive bumps get a human/CI check before merge. The top-level minimumReleaseAge still gates direct dependency updates.

@vdusek vdusek added t-tooling Issues with this label are in the ownership of the tooling team. adhoc Ad-hoc unplanned task added during the sprint. labels Jun 18, 2026
@vdusek vdusek self-assigned this Jun 18, 2026
@github-actions github-actions Bot added this to the 143rd sprint - Tooling team milestone Jun 18, 2026
@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.97%. Comparing base (970f93b) to head (3d203b2).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1978      +/-   ##
==========================================
+ Coverage   92.94%   92.97%   +0.02%     
==========================================
  Files         167      167              
  Lines       11737    11737              
==========================================
+ Hits        10909    10912       +3     
+ Misses        828      825       -3     
Flag Coverage Δ
unit 92.97% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek closed this Jun 18, 2026
@B4nan

B4nan commented Jun 18, 2026

Copy link
Copy Markdown
Member

This drops automerge/automergeType and sets minimumReleaseAge: "0 days" on lockFileMaintenance, so the refresh opens a regular PR for review instead: the PR reliably appears (no stuck stability check) and the transitive bumps get a human/CI check before merge. The top-level minimumReleaseAge still gates direct dependency updates.

This is a bad idea, the whole point of the min release age dance (or at least the main issue we are trying to solve) is that we don't want to run fresh deps through our CI (so we can't expose our secrets that are available in the workflows). So unless uv would fail to install the deps, we can't go this way.

Lockfile maintenance itself is pretty simple task (drop old lockfile, generate new one, open a PR), I am sure we could craft our own scheduled workflow to do this, if renovate doesn't support uv properly just yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants