Vocabulary: Switch dependabot PRs to monthly cycle

Created on 10 Sep 2020  路  6Comments  路  Source: creativecommons/vocabulary

We should change the interval of our dependabot PRs to monthly, to reduce the frequency with which we need to merge them.


Original discussion below:

Problem

We should automatically merge our dependabot PRs with passing checks, using the merge me GitHub Action.

This will cut down considerably on developer time each sprint.

Implementation

  • [ ] I would be interested in implementing this feature.
Hacktoberfest good first issue help wanted improvement low dx

Most helpful comment

@dhruvkb Agreed. Like I said on the vocab call I think switching the interval to monthly like @nimishbongale is the best path forward.

All 6 comments

@zackkrida

Sometimes, it so happens with dependabot that it says code coverage - unknown in its PR. I'm not sure if this affects anything at all, but this deters me from auto-merging a dependabot PR (in my other personal repo's).

Although this may not be the orthodox way of solving this problem, I would suggest:

  • Changing the frequency of dependabot PR's : Although it seems counter-intuitive, we could change the interval: "weekly" attribute in the .github/dependabot.yml to monthly. Most OSS organisations prefer having it monthly (afaik).
  • Limiting the PR's raised : Dependabot raises 5 PR's per package manager atm. We could limit that as well, so it would raise 5 or less than 5 PR's in the specified package manager only.

@nimishbongale I'm a little skeptical of auto-merging too. The fact that our deployment process is manual makes me less skeptical, but I'll think about it more. I think your suggestions are good compromises.

If we had automated visual testing, merging automatically would not be much of a question. But even in manual deployment, it isn't feasible to individually verify that everything is working as before (and this infeasibility only increases with more components being added to the package).

@dhruvkb Agreed. Like I said on the vocab call I think switching the interval to monthly like @nimishbongale is the best path forward.

I can take this on. Is this change's scope only PRs in vocabulary or all CC repos?

@akmadian

I think it was discussed particularly to do with vocabulary, as I'm not sure about the others (if they want the frequency to be changed ie.)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

zackkrida picture zackkrida  路  6Comments

zackkrida picture zackkrida  路  5Comments

zackkrida picture zackkrida  路  3Comments

annatuma picture annatuma  路  5Comments

rajat2502 picture rajat2502  路  6Comments