Skip to content

ci(all-features): correct protoc's semver directive field for renovate#4770

Closed
rami3l wants to merge 1 commit intorust-lang:mainfrom
rami3l:ci/protoc-renovate-semver
Closed

ci(all-features): correct protoc's semver directive field for renovate#4770
rami3l wants to merge 1 commit intorust-lang:mainfrom
rami3l:ci/protoc-renovate-semver

Conversation

@rami3l
Copy link
Member

@rami3l rami3l commented Mar 23, 2026

Clearly renovate wasn't happy about dfc31fe:

{
  currentValue: "34.1",
  datasource: "github-releases",
  depName: "protocolbuffers/protobuf",
  packageName: "protocolbuffers/protobuf",
  replaceString: '# renovate: datasource=github-releases depName=protocolbuffers/protobuf versioning=semver\n          PROTOBUF_VERSION: "34.1"\n',
  skipReason: "invalid-value",
  versioning: "semver",
  warnings: [],
  updates: [],
}

The most possible reason seems to be that semver actually means fully accurate semver and nothing else. Hopefully semver-coerced will make it happy again.

I also took the chance to clarify #4767 (comment).

References:

@rami3l rami3l marked this pull request as ready for review March 23, 2026 14:50
@rami3l
Copy link
Member Author

rami3l commented Mar 24, 2026

Closed in favor of #4773.

@rami3l rami3l closed this Mar 24, 2026
Sign up for free to 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