Skip to content

Add dependabot config #31

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Aug 1, 2024

This will batch all updates into a single PR once a month.

@tamird
Copy link
Contributor Author

tamird commented Feb 21, 2025

@HighCommander4 now that we're mostly up to date, this can be considered to help keep us there.

@HighCommander4 HighCommander4 self-requested a review February 21, 2025 02:13
@tamird
Copy link
Contributor Author

tamird commented Mar 24, 2025

@HighCommander4 mind having a look? low hanging fruit here.

@HighCommander4
Copy link
Contributor

HighCommander4 commented Mar 25, 2025

Question: does dependabot distinguish between dependency updates in the following categories:

A. Updates that fix security vulnerabilities
B. Updates that break API (hopefully no overlap with A)
C. Updates that neither fix security vulnerabilities nor break API

My reason for asking is that I'd like updates in category A, but not updates in category B. I could go either way on updates in category C (I would probably pass on them if given the choice, since they could still introduce regressions, but if e.g. there's no way to distinguish between A and C then I'd take them).

@tamird
Copy link
Contributor Author

tamird commented Mar 25, 2025

There are a lot of configuration knobs but in general dependabot can't know which update breaks API and which doesn't. You can specify which updates you want in terms of semver - that's about it.

My suggestion is to start with this and customize as we go.

@HighCommander4
Copy link
Contributor

HighCommander4 commented Mar 26, 2025

You can specify which updates you want in terms of semver - that's about it.

That sounds like it should be sufficient for excluding API-breaking updates, since updates that break API need to bump the major version in semver.

So, can we configure it to exclude major version bumps, e.g. something like this?

@tamird
Copy link
Contributor Author

tamird commented Mar 26, 2025

Yes, we can - but are you sure that's what you want? Some portion of even those updates will be OK as they won't break APIs we actually use.

@HighCommander4
Copy link
Contributor

Yes, we can - but are you sure that's what you want?

Speaking only for what I'm personally willing to take on as a maintainer: yes. I'm fine with monthly PRs that update dependencies but do not require updating API usage. I do not have the bandwidth to deal with potentially having to update API usage on a monthly cadence.

If another maintainer (such as @hokein) is willing to take on the latter, or if you're interested in becoming a maintainer and taking that on, we could take a different course of action.

@tamird
Copy link
Contributor Author

tamird commented Mar 27, 2025

I'd be willing to help with maintenance.

@HighCommander4
Copy link
Contributor

I'd be willing to help with maintenance.

Thanks!

@hokein, assuming you're on board with that, could you add @tamird as a project member please?

This will batch all updates into a single PR once a month.
@tamird
Copy link
Contributor Author

tamird commented Apr 9, 2025

Gentle ping on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants