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

dependabot.yml: remove time and timezones. #35

Closed
wants to merge 1 commit into from

Conversation

MikeMcQuaid
Copy link
Member

Let's be consistent across the Homebrew organisation here.

Let's be consistent across the Homebrew organisation here.
@Bo98
Copy link
Member

Bo98 commented May 24, 2024

14:30 usually overlaps with the usual ops maintenance slot and across the most timezones which I think is better in case a deploy goes wrong (we don't have full CI testing here). Not entirely convinced it needs to sync with all repos unless you are planning to update Homebrew/.github and/or Homebrew/brew to use grouped updates and the npm stuff too?

But if you feel stronger with Dependabot picking its own 24 hour interval then sure I guess. It's 24 hours either way.

Note that this does not affect Dependabot's security update system as it doesn't read the time settings in the config, in case that's what you were concerned about. Those are triggered by GitHub security notifications.

@MikeMcQuaid
Copy link
Member Author

Ok, fine with me, thanks!

@MikeMcQuaid MikeMcQuaid deleted the dependabot_timezones branch May 24, 2024 18:08
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants