Skip to content

feat: add generic bumpVersions option#34023

Merged
secustor merged 24 commits intorenovatebot:mainfrom
secustor:feat/bump-versions
May 6, 2025
Merged

feat: add generic bumpVersions option#34023
secustor merged 24 commits intorenovatebot:mainfrom
secustor:feat/bump-versions

Conversation

@secustor
Copy link
Copy Markdown
Member

@secustor secustor commented Feb 4, 2025

Changes

Implements bumpVersions which leans the semantics of our custom managers

Context

Closes #28045

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

https://github.com/secustor/renovate-bump-versions

@secustor secustor requested review from rarkins and viceice February 4, 2025 09:56
Comment thread lib/config/options/index.ts
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.ts
Comment thread lib/workers/repository/update/branch/index.ts Outdated
Copy link
Copy Markdown
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but does it bump the version in every update PR? I see no filter to limit it.

As far as i can see if i configure it for a specific file (eg Chart.yaml for values.yaml updates), then it will bump it even for updates on workflows?
Shouldn't we limit this to specifix updates somehow? can it be used in package rules? i don't see it in docs.

Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
@viceice
Copy link
Copy Markdown
Member

viceice commented Mar 5, 2025

will be conflicted with #34615

@secustor secustor force-pushed the feat/bump-versions branch from 80446ba to c306cb7 Compare March 5, 2025 19:13
@secustor
Copy link
Copy Markdown
Member Author

secustor commented Mar 5, 2025

LGTM, but does it bump the version in every update PR? I see no filter to limit it.

As far as i can see if i configure it for a specific file (eg Chart.yaml for values.yaml updates), then it will bump it even for updates on workflows? Shouldn't we limit this to specifix updates somehow? can it be used in package rules? i don't see it in docs.

This would be the case if you configure it in the root.
For such cases people can use packageRules e.g. with matchFileNames. I have added an example to the reproduction

@secustor secustor requested a review from viceice March 5, 2025 19:14
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
@secustor secustor enabled auto-merge March 6, 2025 23:56
@secustor secustor requested a review from viceice March 6, 2025 23:57
Comment thread docs/usage/configuration-options.md
Comment thread lib/config/options/index.ts
Comment thread docs/usage/configuration-options.md Outdated
Comment thread docs/usage/configuration-options.md Outdated
Comment thread lib/config/options/index.ts Outdated
Comment thread docs/usage/configuration-options.md Outdated
Comment thread docs/usage/configuration-options.md Outdated
Comment thread lib/config/validation.ts Outdated
Copy link
Copy Markdown
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwisew LGTM

Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
@secustor secustor requested a review from viceice May 4, 2025 23:24
@secustor
Copy link
Copy Markdown
Member Author

secustor commented May 4, 2025

I had to change to change the logic a bit, to ensure the order of FileChanges

Comment thread docs/usage/configuration-options.md Outdated
Comment thread lib/config/validation.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.spec.ts Outdated
Comment thread lib/workers/repository/update/branch/bump-versions.ts
secustor and others added 3 commits May 6, 2025 14:52
@secustor secustor requested a review from viceice May 6, 2025 13:06
@secustor secustor added this pull request to the merge queue May 6, 2025
Merged via the queue into renovatebot:main with commit c4a2862 May 6, 2025
40 checks passed
@secustor secustor deleted the feat/bump-versions branch May 6, 2025 13:24
@renovate-release
Copy link
Copy Markdown

🎉 This PR is included in version 40.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Comment thread docs/usage/configuration-options.md
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[META] Allow to bump generic version references via new bumpVersions config option

5 participants