-
Notifications
You must be signed in to change notification settings - Fork 873
ci: Update Renovate bot configuration #3307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the Renovate bot configuration to refine dependency update behavior across the repository. It removes the "feature/geti-inspect" branch from base branches and adds several new rules to control automatic updates for different dependency types.
Changes:
- Removed "feature/geti-inspect" from base branches
- Added rules to disable major/minor updates for Python and Node Docker base images
- Disabled automatic npm and ruff updates with manual override capability
- Added restrictions for GitHub Actions updates on non-main branches
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
adcabcf to
51b5a50
Compare
|
Codecov Reportβ All modified and coverable lines are covered by tests. π’ Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
AlexanderBarabanov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, Ivan!
Could you please also test proposed config in a fork?
|
|
||
| // [!] While updating the Renovate config, test changes on your own fork. | ||
| // 1. Modify the Renovate configuration, which is located in .github/renovate.json5 and push your changes to the default branch of your fork. | ||
| // 2. Enable the Renovate GitHub app in your GitHub account. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we somehow group minor/patch Python deps upgrades (see p.2 in #3300)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, also have added grouping for npm updates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
7a43181 to
174188e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| enabled: true, | ||
| matchManagers: ["pep621"], | ||
| groupName: "Python dependencies", | ||
| groupSlug: "python-dependencies", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we group only minor/patch upgrades?
Major upgrades may introduce significant changes and potentially may break CI - it is better to process each major upgrade separately while grouping all non-major upgrades.
I propose to have TWO rules:
- bi-weekly group update for non-major - 1 PR
- monthly individual upgrades for major - a separate PR for each upgrade
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All right, re-grouping as suggested
| { | ||
| enabled: true, | ||
| matchManagers: ["npm"], | ||
| matchUpdateTypes: ["minor", "patch"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add schedule parameter (bi-weekly) and a separate rule for major upgrades - non-grouped with monthly cadence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have added schedule for grouped non-major updates and monthly major update with one PR per change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
β¦ts (#3308) * Update renovate config Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com> * Initial plan Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com> * Fix spelling: change 'digital' to 'digest' in comments Co-authored-by: ivanzati <96478173+ivanzati@users.noreply.github.com> Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com> --------- Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com> Signed-off-by: Ivan Zakharov <ivan.zakharov@intel.com> Co-authored-by: Zakharov, Ivan <ivan.zakharov@intel.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: ivanzati <96478173+ivanzati@users.noreply.github.com> Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
* Update renovate config * Initial plan * Fix spelling: change 'digital' to 'digest' in renovate config Co-authored-by: ivanzati <96478173+ivanzati@users.noreply.github.com> --------- Co-authored-by: Zakharov, Ivan <ivan.zakharov@intel.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: ivanzati <96478173+ivanzati@users.noreply.github.com> Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Ivan Zakharov <ivan.zakharov@intel.com> Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
Co-authored-by: Alexander Barabanov <97449232+AlexanderBarabanov@users.noreply.github.com> Signed-off-by: Ivan Zakharov <ivan.zakharov@intel.com> Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
Co-authored-by: Alexander Barabanov <97449232+AlexanderBarabanov@users.noreply.github.com> Signed-off-by: Ivan Zakharov <ivan.zakharov@intel.com> Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
Signed-off-by: Zakharov, Ivan <ivan.zakharov@intel.com>
9c45ebd to
500905d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 5 comments.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
π Description
β¨ Changes
Select what type of change your PR is:
β Checklist
Before you submit your pull request, please make sure you have completed the following steps:
For more information about code review checklists, see the Code Review Checklist.