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

Add dependabot.yml to reduce manual work #7100

Merged
merged 3 commits into from
Sep 12, 2024

Conversation

birkskyum
Copy link
Contributor

@birkskyum birkskyum commented Aug 13, 2024

I noticed these recent commits:

This PR adds a dependabot that'll automate the bumping of minor/patch versions to keep them up to date, with release notes etc. (example here), so that the only work left for maintainers is to merge.

It'll not bump major versions of deps, which could lead to breaking changes.

There will be at most 1 open pr. per lockfile (so two, root+stackgl) with current settings.

It'll not auto merge unless this github workflow is also added. We actually do auto merge in maplibre, because the CI is the bar it has to meet anyway, and it's honestly quite a relief to be able to focus on the areas that needs attention.

@gvwilson gvwilson added community community contribution P2 needed for current cycle fix fixes something broken labels Aug 16, 2024
@gvwilson gvwilson added P1 needs immediate attention and removed P2 needed for current cycle labels Aug 21, 2024
@archmoj
Copy link
Contributor

archmoj commented Sep 9, 2024

Thanks very much @birkskyum for this PR.
To avoid potential merge conflicts from various dependencies namely webpack with the esbuild PR, I prefer to merge this after #6909 is merged.

@birkskyum
Copy link
Contributor Author

that makes sense - looking forward to the esbuild setup, it feels quite productive.

@birkskyum
Copy link
Contributor Author

birkskyum commented Sep 11, 2024

And just to clarify, it'll not make any conflicts unless the PRs are manually merged, and it's set to open max 2.

@archmoj
Copy link
Contributor

archmoj commented Sep 12, 2024

Thanks very much for the PR.

@archmoj archmoj merged commit 02ef675 into plotly:master Sep 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community community contribution fix fixes something broken P1 needs immediate attention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants