chore(fix): prevent renovate from using the canonical version for bindings go module versions by Skarlso · Pull Request #2670 · open-component-model/open-component-model

Skarlso · 2026-05-28T10:56:00Z

What this PR does / why we need it

So this was a bit more involved actually... :D

Renovate created this pr: #2664

The solution in the end is to tell renovate to NOT use the canonical tag as a tag for the submodules.

Which issue(s) this PR fixes

Testing

How to test the changes

Verification

I have added/updated tests for my changes (see Test Requirements)
Tests pass locally (task test and task test/integration if applicable)
If touching multiple modules, go work is enabled (see go.work)
My changes do not decrease test coverage
I have tested the changes locally by running ocm

netlify · 2026-05-28T10:56:05Z

✅ Deploy Preview for ocm-website ready!

Name	Link
🔨 Latest commit	`e25bcc1`
🔍 Latest deploy log	https://app.netlify.com/projects/ocm-website/deploys/6a193d4c23bdfb0008387c58
😎 Deploy Preview	https://deploy-preview-2670--ocm-website.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

coderabbitai · 2026-05-28T10:56:16Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6426b10f-a203-4bb0-b732-e638e1a33dcc

📥 Commits

Reviewing files that changed from the base of the PR and between 1f8b602 and 329d97b.

📒 Files selected for processing (1)

.github/renovate.json5

🚧 Files skipped from review as they are similar to previous changes (1)

.github/renovate.json5

📝 Walkthrough

Walkthrough

Renovate packageRules for ocm.software/open-component-model/** (gomod) now restrict matching to v0.0.0- prerelease patterns via matchCurrentValue and allowedVersions; prior schedule and minimumReleaseAge settings were removed.

Changes

Renovate OCM update configuration

Layer / File(s)	Summary
OCM version-specific update rule `.github/renovate.json5`	The `packageRules` entry for `ocm.software/open-component-model/**` now matches current values starting with `v0.0.0-` using `matchCurrentValue` and limits `allowedVersions` to `^v0\.0\.`; scheduling and `minimumReleaseAge` fields were removed while `enabled: false` is preserved.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

open-component-model/open-component-model#2450: Earlier changes to .github/renovate.json5 adjusting Renovate packageRules for gomod/OCM update matching and scheduling.

Suggested reviewers

jakobmoellerdev

Suggested labels

size/xs

Poem

🐰 I hopped through rules with nimble feet,
v0.0.0- now snug and neat,
Renovate reads the modest line,
Skipping updates, all fine and fine,
A carrot-coded config treat.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: updating Renovate configuration to prevent using canonical version tags for bindings Go module versions.
Description check	✅ Passed	The description is related to the changeset, explaining why the Renovate configuration was updated and referencing the original problem (PR `#2664`).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai · 2026-05-28T10:57:28Z

Actionable comments posted: 0

jakobmoellerdev · 2026-05-28T11:06:24Z

This will effectively break all monorepo updates. How would you expect that no bumps are the correct expected behavior?

Skarlso · 2026-05-28T11:16:22Z

This will effectively break all monorepo updates. How would you expect that no bumps are the correct expected behavior?

I don't think it will. 🤔

Skarlso · 2026-05-28T11:20:14Z

Ah, I see. We did have proper sha updates.

Skarlso · 2026-05-28T11:21:46Z

Ahhh I see. I think my first instinct will be right after all. Let me update this.

… version On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com> Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>

Skarlso · 2026-05-28T11:36:45Z

Mmmm.... my test still returns the canonical tag... hold on here.

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/renovate.json5:
- Around line 182-187: The Renovate rule for the gomod package pattern
(ocm.software/open-component-model/**) currently uses matchCurrentValue:
'/^v0\\.0\\.0-/' but restricts disabling to only matchUpdateTypes:
['major','minor','patch']; remove the matchUpdateTypes entry (or replace it with
the complete list you intend to block) so the v0.0.0-* pseudo-version rule
applies to all update types; update the rule where matchCurrentValue and
matchUpdateTypes appear in the .github/renovate.json5 config accordingly and run
a Renovate dry-run to confirm v0.0.0-* deps are no longer updated for any
updateType.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b52aedfd-3e53-40dc-95ec-39fa7a719eb2

📥 Commits

Reviewing files that changed from the base of the PR and between a58ecbf and 0ad6b5f.

📒 Files selected for processing (1)

.github/renovate.json5

On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com> Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>

…nt-model into fix-renovate-rc-tag

coderabbitai · 2026-05-28T11:59:59Z

Actionable comments posted: 0

On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com> Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>

frewilhelm

lgtm, but i would really love a comment explaining what is happening, so we do not have to wonder in the future

Skarlso · 2026-05-29T07:15:36Z

@frewilhelm I'll add one in my next pr dealing with the release flow. :) Don't want to dismiss the 👍 s :D

Skarlso requested a review from a team as a code owner May 28, 2026 10:56

github-actions Bot added kind/chore chore, maintenance, etc. size/xs Extra small labels May 28, 2026

github-actions Bot added the component/github-actions Changes on GitHub Actions or within `.github/` directory label May 28, 2026

chore(fix): tell renovate to not fetch the canonical tag as go module…

fb77261

… version On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com> Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>

Skarlso force-pushed the fix-renovate-rc-tag branch from a58ecbf to fb77261 Compare May 28, 2026 11:28

github-actions Bot added size/s Small and removed size/xs Extra small labels May 28, 2026

Merge branch 'main' into fix-renovate-rc-tag

0ad6b5f

piotrjanik previously approved these changes May 28, 2026

View reviewed changes

piotrjanik self-requested a review May 28, 2026 11:36

coderabbitai Bot reviewed May 28, 2026

View reviewed changes

Comment thread .github/renovate.json5 Outdated

Skarlso added 2 commits May 28, 2026 13:57

use allowedVersions instead for matching rules to strict versions

b760e6a

On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com> Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>

Merge branch 'fix-renovate-rc-tag' of github.com:Skarlso/open-compone…

1f8b602

…nt-model into fix-renovate-rc-tag

Skarlso dismissed piotrjanik’s stale review via 1f8b602 May 28, 2026 11:57

Skarlso changed the title ~~chore(fix): remove the usage of direct resolution for renovate~~ chore(fix): prevent renovate from using the canonical version for bindings go module versions May 28, 2026

jakobmoellerdev reviewed May 28, 2026

View reviewed changes

Comment thread .github/renovate.json5 Outdated

relax the version constraint

8c8a7b1

On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com> Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>

Skarlso mentioned this pull request May 28, 2026

Follow-up optimizations and fixes of existing issues for the release refactor open-component-model/ocm-project#1076

Closed

8 tasks

jakobmoellerdev reviewed May 28, 2026

View reviewed changes

Comment thread .github/renovate.json5

jakobmoellerdev approved these changes May 28, 2026

View reviewed changes

frewilhelm approved these changes May 29, 2026

View reviewed changes

Merge branch 'main' into fix-renovate-rc-tag

329d97b

Merge branch 'main' into fix-renovate-rc-tag

e25bcc1

Skarlso merged commit 4d9d16d into open-component-model:main May 29, 2026
22 checks passed

matthiasbruns mentioned this pull request May 29, 2026

fix(deps): update ocm monorepo go module dependencies #2664

Merged

1 task

coderabbitai Bot mentioned this pull request Jun 8, 2026

fix: adjust config and renovate run to renovate fix #2782

Merged

Conversation

Skarlso commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What this PR does / why we need it

Which issue(s) this PR fixes

Testing

How to test the changes

Verification

Uh oh!

netlify Bot commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for ocm-website ready!

Uh oh!

coderabbitai Bot commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Suggested labels

Poem

Uh oh!

coderabbitai Bot commented May 28, 2026

Uh oh!

jakobmoellerdev commented May 28, 2026

Uh oh!

Skarlso commented May 28, 2026

Uh oh!

Skarlso commented May 28, 2026

Uh oh!

Skarlso commented May 28, 2026

Uh oh!

Skarlso commented May 28, 2026

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

coderabbitai Bot commented May 28, 2026

Uh oh!

Uh oh!

Uh oh!

frewilhelm left a comment

Choose a reason for hiding this comment

Uh oh!

Skarlso commented May 29, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Skarlso commented May 28, 2026 •

edited

Loading

netlify Bot commented May 28, 2026 •

edited

Loading

coderabbitai Bot commented May 28, 2026 •

edited

Loading