-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Workflow to automerge Dependabot PRs #35
Comments
Additionally these are mostly noise and I would say iff the CI check passes they should be automatically closed. A user can update the version manually if they need to pickup some newer dependencies for some API, and the failing builds which are the "🚩something is probably broken here" would then be left for a maintainer to address some source (or binary) incompatibilities. |
I just use this in my plugins: https://github.com/jenkinsci/slack-plugin/blob/master/.github/renovate.json |
openrewrite/rewrite-jenkins#20 (comment) yes that is an alternative, though enabling automerge from DB is straightforward enough and probably more familiar to most developers. On a tangential note, I am not sure what to do about the dozens of PRs like jenkinsci/workflow-api-plugin#361 as jenkinsci/plugin-pom#1004 is not yet appropriate for most plugins. We could recommend that the DB config for plugins ignore major version bumps for |
I don't think there's a great way to solve it for dependabot apart from updating every repository to ignore v5 or just waiting a bit (few weeks?) and updating baselines (open-rewrite?) For renovate, setting up an shared preset could be done and plugins changed to use that so in future we could have less effort for something like this happening: https://docs.renovatebot.com/config-presets/ |
(One-off example FTR: jenkinsci/archetypes#763) |
Manually merging things like jenkinsci/workflow-api-plugin#359 is very tedious if you are responsible for a lot of plugin repositories. Something like jenkinsci/build-token-root-plugin@0c8842a is a lot more convenient. Does it make sense to make a reusable workflow out of this?
Ideally it would be more selective about which dependencies to merge automatically and which to leave to human review.
plugin-pom
andgit-changelist-maven-extension
are almost always safe.bom-*
like in jenkinsci/build-token-root-plugin#171 is potentially unwanted, since it can gratuitously pull in newer deps on other plugins that can make certain backporting tasks harder. Dependencies on non-BOM plugins would probably not be wanted for automerge for the same reason. Maven plugins are typically fine to automerge. Third-party library JARs could go either way. Is there some include/exclude pattern which would be sufficiently universal and uncontroversial?The text was updated successfully, but these errors were encountered: