Skip to content

ci(dependabot): change to monthly interval #774

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

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

ErikSchierboom
Copy link
Member

@ErikSchierboom ErikSchierboom commented Sep 4, 2024

This PR changes the interval for dependabot updates to monthly.

Right now, the interval is usually set to daily, which results in (almost) daily notifications for Exercism maintainers, and frequent Docker updates to all tooling repos.

To reduce the maintenance burden, we are lowering the frequency to monthly. Due to Exercism's setup, it is unlikely that dependabot issues in our repositories need to be handled with any urgency, so we consider this to be a wise tradeoff.

For maintained track repos, maintainers can choose to close this PR unmerged. But for tooling repos and unmaintained tracks, we are requiring it to be merged.

Thanks!

@ErikSchierboom ErikSchierboom requested a review from a team as a code owner September 4, 2024 11:27
@ErikSchierboom ErikSchierboom added the x:size/tiny Tiny amount of work label Sep 4, 2024
@colinleach
Copy link
Contributor

colinleach commented Sep 7, 2024

I'm unsure what to do with this and similar PRs. Merging is blocked, pending approval by maintainers-admin, but apparently that is still just Erik. Should this now be @exercism/cross-track-maintainers? If so, @IsaacG is part of that.

@colinleach colinleach self-assigned this Sep 7, 2024
@colinleach
Copy link
Contributor

It lets me add myself as a reviewer, but I can't remove Erik.

@colinleach colinleach removed their assignment Sep 7, 2024
@BNAndras
Copy link
Member

BNAndras commented Sep 7, 2024

That seems odd. I’ve merged quite a few of these PRs as a cross-track-maintainer member. What if we make a new PR with the changes from this PR and then close this one?

@colinleach
Copy link
Contributor

Sure, whatever works.

Unfortunately this may also be necessary for PRs #767, #772, #773, #775. I'll make new PRs.

@IsaacG
Copy link
Member

IsaacG commented Sep 7, 2024

I asked about that here (restricted thread): https://forum.exercism.org/t/maintained-vs-single-maintained-vs-unmaintained-tracks/12636/69

@colinleach
Copy link
Contributor

Did you get a reply? (I don't have read access to this thread)

@IsaacG
Copy link
Member

IsaacG commented Sep 7, 2024

Did you get a reply? (I don't have read access to this thread)

Not yet. Jeremy and Erik are usually less active on the weekend.

@ErikSchierboom ErikSchierboom merged commit ad17585 into main Sep 9, 2024
1 check passed
@ErikSchierboom ErikSchierboom deleted the dependabot-interval branch September 9, 2024 06:23
@ErikSchierboom
Copy link
Member Author

See #778

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:size/tiny Tiny amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants