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

.github: change dependabot interval to daily #15651

Merged
merged 1 commit into from
Apr 12, 2021

Conversation

aanm
Copy link
Member

@aanm aanm commented Apr 12, 2021

Signed-off-by: André Martins andre@cilium.io

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm requested review from twpayne and tklauser April 12, 2021 12:53
@aanm aanm requested a review from a team as a code owner April 12, 2021 12:53
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 12, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Apr 12, 2021
@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Apr 12, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 12, 2021
Copy link
Contributor

@twpayne twpayne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but are you sure you want to do this? Dependabot can get pretty spammy especially if a dependency does a flurry of updates.

@tklauser
Copy link
Member

LGTM, but are you sure you want to do this? Dependabot can get pretty spammy especially if a dependency does a flurry of updates.

We still restrict the number of open PRs perpackage ecosystem to 1. So I guess this will at least prevent dependabot from spamming us with a huge number of updates at once.

@aanm
Copy link
Member Author

aanm commented Apr 12, 2021

Let's see how it goes.

@aanm aanm merged commit eaa0172 into cilium:master Apr 12, 2021
1.10.0 automation moved this from In progress to Done Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants