Skip to content

chore: bump the self-referential apis/v2 requirement at release time - #99

Open
phisco wants to merge 1 commit into
mainfrom
template-apis-version-bump
Open

chore: bump the self-referential apis/v2 requirement at release time#99
phisco wants to merge 1 commit into
mainfrom
template-apis-version-bump

Conversation

@phisco

Copy link
Copy Markdown
Contributor

The root go.mod requires github.com/crossplane/crossplane/apis/v2 and replaces it with ./apis. The replace only applies in the main module, so for anything importing github.com/crossplane/crossplane/v2 that requirement is the minimum apis version resolved from the proxy — and a stale value publishes a module that does not compile. crossplane/v2@v2.4.0 recorded apis v2.3.4, which lacks v1.RuntimeActive and friends.

Renovate only bumps this on main; release branches have non-security Go updates disabled, so it has to be part of the release checklist. Folded into the runtime-bump PRs already there, so it costs no extra steps. The Tag workflow creates apis/vX.Y.Z at the same commit as vX.Y.Z, so the PR can name a tag that does not exist yet.

The root go.mod requires github.com/crossplane/crossplane/apis/v2 and
replaces it with ./apis. The replace only applies in the main module, so for
anything importing github.com/crossplane/crossplane/v2 that requirement is
the minimum apis version resolved from the proxy — and a stale value
publishes a module that does not compile. crossplane/v2@v2.4.0 recorded apis
v2.3.4, which lacks v1.RuntimeActive and friends.
Renovate only bumps this on main; release branches have non-security Go
updates disabled, so it has to be part of the release checklist. Folded into
the runtime-bump PRs already there, so it costs no extra steps. The Tag
workflow creates apis/vX.Y.Z at the same commit as vX.Y.Z, so the PR can
name a tag that does not exist yet.
Signed-off-by: Philippe Scorsolini <5697904+phisco@users.noreply.github.com>
Sign up for freeto 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.

1 participant

@phisco