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

Reduce frequency of dependency update PRs #3077

Open
ST-DDT opened this issue Aug 26, 2024 · 3 comments
Open

Reduce frequency of dependency update PRs #3077

ST-DDT opened this issue Aug 26, 2024 · 3 comments
Labels
c: infra Changes to our infrastructure or project setup p: 1-normal Nothing urgent s: needs decision Needs team/maintainer decision
Milestone

Comments

@ST-DDT
Copy link
Member

ST-DDT commented Aug 26, 2024

Currently we receive dependency update PRs once a week.
But it sometimes takes half a week to review and merge them and during that time I get a lot of emails regarding their updates.
So I propose to reduce the updates to once per month and on demand before updates.

Lets say we have 5 dependency PRs per week on average (this week it have been six).
This produces 38 mails per week = 152 mails/month.
If we reduce it to monthly and we assume 7 PRs. Then we get this number down to 53.
If we manually trigger the dependency updates (sequential, 7 PRs), this number goes down to 35.

As an additional benefit, this removes a lot of dependency updates from our release notes.
What do you think?

@ST-DDT ST-DDT added p: 1-normal Nothing urgent s: needs decision Needs team/maintainer decision c: infra Changes to our infrastructure or project setup labels Aug 26, 2024
@ST-DDT ST-DDT added this to the vAnytime milestone Aug 26, 2024
@matthewmayer
Copy link
Contributor

Strong support. I find the dependency notifications very spammy

And given faker only has devDependencies I don't think there is such an urgency to always be on the bleeding edge version.

@xDivisionByZerox
Copy link
Member

This sounds reasonable to me. I'd be in favor of manual, sequential PRs to further reduce the noise from renovate rebases as described by ST-DDT.

@Shinigami92
Copy link
Member

Team-Decision:

For now we want to try out to simply ignore notifications of Renovate (or GitHub in general), if we do not feel like we have the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: infra Changes to our infrastructure or project setup p: 1-normal Nothing urgent s: needs decision Needs team/maintainer decision
Projects
None yet
Development

No branches or pull requests

4 participants