Skip to content

fix(renovate): normalize SHA-pin comments to major-version-only - #20

Merged
wgordon17 merged 1 commit into
gordon-code:mainfrom
wgordon17:fix/extract-major-version-only
Jul 28, 2026
Merged

fix(renovate): normalize SHA-pin comments to major-version-only#20
wgordon17 merged 1 commit into
gordon-code:mainfrom
wgordon17:fix/extract-major-version-only

Conversation

@wgordon17

Copy link
Copy Markdown
Member

Summary

  • Renovate's native github-actions manager writes major-only version comments (e.g. v7) when bumping SHA-pinned actions in root workflow files, but the custom regex customManagers for the .jinja template sources resolved the full github-releases release tag (e.g. v7.0.0) for the same dependency — causing the consistency check to fail whenever a template's SHA-pin/flake-input comment diverges from its already-bumped root counterpart (seen on PR chore(deps): update dependency actions/checkout to v7 #19)
  • Adds extractVersionTemplate: "^(?<version>v\\d+)" to both the SHA-pinned-actions and Nix-flake-input customManagers so they normalize to the same major-only granularity as the native manager going forward
  • Verified: regenerated .github/renovate.json validates cleanly against the fleet's actual pinned Renovate version (43.150.0)

Renovate's native github-actions manager writes major-only comments
(e.g. "v7") when bumping the root workflow files, but the custom
regex customManagers for the .jinja template sources resolved the
full github-releases tag (e.g. "v7.0.0") for the same dependency --
same logical bump, two different comment formats, causing the
consistency check to fail whenever the SHA-pin/flake-input comment
in a template file diverges from its already-bumped root counterpart
(seen on PR gordon-code#19). extractVersionTemplate normalizes both managers to
the same major-only granularity going forward.
@wgordon17
wgordon17 marked this pull request as ready for review July 28, 2026 19:18
@wgordon17
wgordon17 merged commit fb5e3ce into gordon-code:main Jul 28, 2026
2 checks passed
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