Skip to content
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

Less frequent, and grouped dependabot updates #1271

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

anderseknert
Copy link
Member

Monthly may seem like an excaggeration, but our only real dependency is OPA, and we update that ourselves anyway. The rest are just peripheral, and security related updates will be suggested immediately regardless.

Monthly may seem like an excaggeration, but our only real dependency
is OPA, and we update that anyway. The rest are just peripheral anyway,
and security related updates will be suggested immediately regardless.

Signed-off-by: Anders Eknert <[email protected]>
@charlieegan3 charlieegan3 merged commit 6716d37 into main Nov 20, 2024
5 checks passed
@charlieegan3 charlieegan3 deleted the less-frequent-dependabot branch November 20, 2024 13:44
@srenatus
Copy link
Member

One disadvantage of groups is that you cannot ignore them through interacting with github PRs: If it opens a PR for yadda, and for some reason you don't want to bump yadda, and you close the PR, dependabot will tell you that "because it's a group, I cannot do anything" and will re-open the same PR on the next schedule.

Without the group, closing the PR would make dependabot ignore that particular version.

Just a small annoyance I wanted to share 😅

@anderseknert
Copy link
Member Author

That's good to know! I don't think we've had any updates that we've not wanted so far, but if that becomes annoying, yeah, let's ditch groups. I want them mainly to avoid having to wait for each PR to be rebuilt after on is merged.

@charlieegan3
Copy link
Member

Agreed, the main pain is the rebase time, not the unwanted GHA updates as things are today.

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.

3 participants