Skip to content

ci(renovate): Run only at nights and only open 10 PRs - #6902

Merged
nickvergessen merged 4 commits into
masterfrom
ci/noid/renovate
Oct 14, 2026
Merged

nickvergessen merged 4 commits into
masterfrom
ci/noid/renovate

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added feature: CI 🏗️ Android Studio, CI, Drone, Beta builds, Alpha builds dependencies labels Oct 13, 2026
@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

📝 Walkthrough

The Renovate configuration now limits concurrent PRs to 10, rebases conflicted branches, schedules updates for 1–4 a.m. daily, and sets a 10-day minimum release age. It groups GitHub Actions updates under “GitHub Actions.”

Priority: ⬇️ Low

Merge Risk: 🔵 Low · up to 0eb4e

Some open Renovate PR branches can still be updated outside the 01:00–04:59 Berlin window. Disable out-of-schedule updates to fully limit update activity to the intended hours.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description only includes the AI disclosure. It omits the required screenshots, TODO, and checklist sections, including test, capability, backport, milestone, and title confirmations. Add the required template sections. Complete the screenshots section or state that screenshots are not applicable. Complete the TODO section and each checklist item. Retain the AI disclosure.
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly describes the main changes: scheduling Renovate runs at night and limiting open pull requests to 10.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Joas Schilling <coding@schilljs.com>

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to GitHub limitations.

⚠️ Outside diff range comments (1)

🟡 Minor · Apply the schedule to existing Renovate branches. · renovate.json5:7-9

renovate.json5:7-9
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Apply the schedule to existing Renovate branches.

updateNotScheduled defaults to true. Renovate can therefore update existing branches outside the configured schedule. An open Renovate PR branch may receive such an update outside 01:00–04:59 Europe/Berlin. With rebaseWhen: "conflicted", this applies when the branch has conflicts, not when it is merely behind.

Set updateNotScheduled to false to prevent these out-of-schedule updates.

Suggested fix
   "timezone": "Europe/Berlin",
+  "updateNotScheduled": false,
   "prConcurrentLimit": 10,

ℹ️ Review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2b14de20-9201-4a36-b7ba-0d901f208ab3
📥 Commits

Reviewing files that changed from the base of the PR and between f8b1bc5 and 0eb4e51.

📒 Files selected for processing: 1
  • renovate.json5

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review.

@AndyScherzinger AndyScherzinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good, thought we do 7 days and exclude our own libs from the cool down?

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen

Copy link
Copy Markdown
Member Author

thought we do 7 days

Tried that, but zizmor raised the warning length recently, so it contains 2 weekends

@nickvergessen
nickvergessen merged commit 196641a into master Oct 14, 2026
9 of 19 checks passed
@nickvergessen
nickvergessen deleted the ci/noid/renovate branch October 14, 2026 06:48
@AndyScherzinger AndyScherzinger added this to the 25.1.0 milestone Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted dependencies feature: CI 🏗️ Android Studio, CI, Drone, Beta builds, Alpha builds

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants