Skip to content

chore: migrate Renovate config preset#836

Open
alceops wants to merge 2 commits into
open-feature:mainfrom
alceops:alce/renovate-config-recommended-20260501
Open

chore: migrate Renovate config preset#836
alceops wants to merge 2 commits into
open-feature:mainfrom
alceops:alce/renovate-config-recommended-20260501

Conversation

@alceops
Copy link
Copy Markdown

@alceops alceops commented May 2, 2026

Summary

  • migrate Renovate from config:base to config:recommended
  • migrate the regex manager block to Renovate custom managers while preserving the existing match/file patterns

Verification

  • python3 -m json.tool renovate.json
  • git diff --check

No runtime or cluster tests were run; this is a Renovate config-only change.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Renovate configuration by migrating from config:base to config:recommended, renaming regexManagers to customManagers, and explicitly setting the customType to regex. A review comment suggests further simplifying the configuration by removing redundant entries like group:recommended and semanticCommits, which are already included in the recommended preset.

Comment thread renovate.json
"$schema": "https://docs.renovatebot.com/renovate-schema.json",
"extends": [
"config:base",
"config:recommended",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The config:recommended preset already includes group:recommended and enables semanticCommits by default. Since you are migrating to this preset, the explicit entries for group:recommended (line 5) and "semanticCommits": "enabled" (line 9) are now redundant. Removing them would simplify the configuration and follow the best practice of keeping Renovate configurations minimal.

@alceops alceops force-pushed the alce/renovate-config-recommended-20260501 branch from 777b368 to 90ee695 Compare May 2, 2026 00:34
Signed-off-by: Alce Ops <alceops@users.noreply.github.com>
@alceops alceops force-pushed the alce/renovate-config-recommended-20260501 branch from 90ee695 to 93d4ad6 Compare May 2, 2026 00:40
Copy link
Copy Markdown
Member

@beeme1mr beeme1mr left a comment

Choose a reason for hiding this comment

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

Thanks @alceops

@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.42%. Comparing base (499661e) to head (45f9abc).
⚠️ Report is 100 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #836       +/-   ##
===========================================
- Coverage   86.51%   69.42%   -17.10%     
===========================================
  Files          19       31       +12     
  Lines        1587     2008      +421     
===========================================
+ Hits         1373     1394       +21     
- Misses        173      561      +388     
- Partials       41       53       +12     

see 26 files with indirect coverage changes

Flag Coverage Δ
unit-tests 69.42% <ø> (∅)

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants