-
Notifications
You must be signed in to change notification settings - Fork 1.4k
🌱 Add dependabot groups. Allow additional patch updates #9263
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
🌱 Add dependabot groups. Allow additional patch updates #9263
Conversation
/lgtm |
LGTM label has been added. Git tree hash: de9657b1ccaab09d8ef23493de1be866e0aea075
|
/hold For squash |
Signed-off-by: killianmuldoon <kmuldoon@vmware.com>
Signed-off-by: killianmuldoon <kmuldoon@vmware.com>
566eb38
to
2c863c2
Compare
/tide merge-method-squash To test for the release notes tool ref #9249 |
/hold cancel |
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.
Is there a way to test this (or a link to PR where you already tested it maybe?) on the fork?
/lgtm
We used everything apart from the groups in CAPV for a while and recently merged the groups there. Not sure if we explicitly tested the groups. But as it's a just GitHub feature that we use, I'm fine with just trying it |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbueringer The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I tested the groups on my own fork, but not on a schedule. I saw a couple of race conditions, but it's an improvement in any case IMO. |
Yup in the meantime we got a bunch of PRs opened and it looks fine to me |
Allow patch updates for the ignored dependencies in our dependabot config and group the
k8s.io
dependencies to reduce the number of individual PRs./area ci