-
Notifications
You must be signed in to change notification settings - Fork 887
chore: group dependencies #13798
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
chore: group dependencies #13798
Conversation
I know we had this before, so I'm curious why it was removed. Was it causing any issues? |
@aslilac we did had them in #8453 but were still not happy with amount of PRs dependabot was creating. So we created a single large group of everything in #10870. This made the PRs unmergeable and was creating PRs with 67 dependencies update in a single PR. |
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.
thanks for explaining, I like this approach 😄
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.
The golang.org/x/*
group seems good to me
This should help reduce dependabot noise. Probably grouping can be further improved.