chore(deps): separate Renovate patch grouping by dependency#24119
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe Renovate configuration is updated to remove the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s Renovate configuration to split dependency update grouping, making Renovate PRs smaller and easier to selectively upgrade (especially for minor updates) while still grouping patch updates for dev dependencies.
Changes:
- Removed the
group:allNonMajorpreset to avoid an overly large combined PR. - Added a package rule to group patch updates for devDependencies into a single Renovate PR.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
commit: |
|
Size Change: 0 B Total Size: 1.48 MB ℹ️ View Unchanged
|
Coverage Report
File CoverageNo changed files found. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/renovate.json5:
- Around line 7-15: The renovate rule uses the legacy key depTypeList which
Renovate no longer recognizes; update the rule that defines groupName 'all patch
devDependencies' / groupSlug 'all-patch-dev-dependencies' to use matchDepTypes
instead of depTypeList so the constraint applies to devDependencies for patch
updates.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
🧪 Playground Preview: https://element-plus.run/?pr=24119 |
|
@btea 佬,这个可以搞吗,这样子可以方便把 dev deps 升级上来。另外一部分会影响产物的 deps 先不动。 |
|
有道理,可以搞 👍 |
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
@thinkasany Thanks for your contribution! ❤️ |

拆分 Renovate 分组,以便于更方便的去升级一些依赖。
before:

group:allNonMajor
太大了,根本没办法操作
after:
可以手动选择一些 minor 去升级,然后 dev 的依赖的 patch 可以合并成一个。
minor 还是拆开按包升级比较好(语义更好,知道哪儿个 PR 调整了哪儿个依赖,修改调整也更方便)。
Summary by CodeRabbit