Skip to content

ci: fix release-please extra-files updater for Cargo.toml#22

Merged
StefanSteiner merged 3 commits into
tableau:mainfrom
StefanSteiner:ssteiner/fix-release-please-toml
May 18, 2026
Merged

ci: fix release-please extra-files updater for Cargo.toml#22
StefanSteiner merged 3 commits into
tableau:mainfrom
StefanSteiner:ssteiner/fix-release-please-toml

Conversation

@StefanSteiner
Copy link
Copy Markdown
Contributor

@StefanSteiner StefanSteiner commented May 18, 2026

Summary

  • Root Cargo.toml: switch to type: "toml" with jsonpath: "$.workspace.package.version" — the previous bare-string and type: "generic" approaches both silently failed because release-please auto-selects GenericToml for .toml files (which uses JSONPath, not annotations).
  • Inter-crate version pins (hyperdb-api-core/Cargo.toml, hyperdb-api/Cargo.toml): switch to block markers (x-release-please-start-version / x-release-please-end) which the Generic updater actually supports (preceding-line annotations don't work).
  • Update the version-consistency CI check to not depend on the removed annotation comment.

This is the third attempt at getting release-please to update Cargo.toml files. The previous two approaches failed because:

  1. Bare string "Cargo.toml" → triggers GenericToml (ignores annotations)
  2. type: "generic" with preceding-line annotation → Generic updater only matches annotations on the same line or via block markers

Test plan

  • Merge this PR
  • Delete release-please--branches--main branch if it exists
  • Run gh workflow run release-please.yml
  • Verify the new release PR includes Cargo.toml version bumps
  • Verify version-consistency check passes

StefanSteiner and others added 3 commits May 18, 2026 13:32
- Root Cargo.toml: use type "toml" with jsonpath
  ($.workspace.package.version) — the GenericToml updater parses TOML
  structurally rather than looking for annotation comments.
- Inter-crate pins: use type "generic" with block markers
  (x-release-please-start-version / x-release-please-end) since the
  Generic updater needs the annotation on/around the version line.
- Remove stale preceding-line annotations that the Generic updater
  doesn't support.
- Update version-consistency CI check to not depend on the removed
  annotation comment.
GITHUB_TOKEN suppresses workflow triggers on events it creates.
Switch to a fine-grained PAT (RELEASE_PLEASE_TOKEN) so release-please
PRs trigger CI and its tags trigger release/npm-publish workflows.

Also documents both the PAT approach (current) and the GitHub App
token alternative in GITHUB_OPERATIONS.md.
@StefanSteiner StefanSteiner merged commit d5d2860 into tableau:main May 18, 2026
10 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