Skip to content

docs: sync README container tags with releases via release-please (CRY-59) - #148

Merged
bougyman merged 2 commits into
mainfrom
CRY-59-sync-readme-container-tags
Aug 18, 2026
Merged

docs: sync README container tags with releases via release-please (CRY-59)#148
bougyman merged 2 commits into
mainfrom
CRY-59-sync-readme-container-tags

Conversation

@bougyman

Copy link
Copy Markdown
Member

Summary

  • Readme.adoc hardcoded :v0 container image tags, but the project is at v1.13.2; :v0 points at long-outdated images
  • ci/publish.sh already generates cascading tags (v1.13.2 → v1.13 → v1), so :v1 is the correct floating tag
  • Adds AsciiDoc document attributes with x-release-please-start-major / x-release-please-start-version markers so release-please updates them automatically on every release going forward
  • Registers Readme.adoc in .release-please-config.json extra-files (same mechanism as app/mix.exs)

Changes

  • Readme.adoc: add :image-major-tag: and :image-pin-example: attributes under release-please markers; add subs="attributes+" to the container source block; replace hardcoded :v0 / :v0.1.2 with attribute references
  • .release-please-config.json: add Readme.adoc as a generic extra-file entry

Test plan

  • CI passes (doc/config only change — no Elixir code touched)
  • Verify :{image-major-tag} renders as :v1 in GitHub's AsciiDoc preview
  • Verify callout numbers <1>/<2> still render correctly with subs="attributes+"
  • Confirm that on the next release-please PR, both Readme.adoc attributes and app/mix.exs version are bumped together

Closes: https://linear.app/the-rubyists/issue/CRY-59

🤖 Generated with Claude Code

…Y-59)

Readme.adoc hardcoded container image tags at :v0 even though the project
has been at v1.x since well before the current v1.13.2. ci/publish.sh
already generates cascading tags (v1.13.2 → v1.13 → v1), so :v0 has
pointed at outdated images.

- Add AsciiDoc document attributes (:image-major-tag, :image-pin-example)
  wrapped in x-release-please-start-major and x-release-please-start-version
  markers so release-please keeps them current automatically on every release.
- Add subs="attributes+" to the container source block so the attributes
  render in GitHub's Asciidoctor view; callout numbers are unaffected.
- Register Readme.adoc in .release-please-config.json extra-files (same
  generic mechanism already used for app/mix.exs).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 18, 2026 17:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

…ticks

AsciiDoc inline code spans (backtick-delimited) suppress attribute
substitution, so `:{image-major-tag}` and `:{image-pin-example}` would
render literally rather than expanding to `:v1` and `:v1.13.2`. Use
literal values in the callout text instead. The source block examples
(inside the [source,sh,subs="attributes+"] block) correctly expand
{image-major-tag} because attribute substitution is explicitly enabled
there. The document attribute definitions and release-please markers at
the top of the file are unaffected.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bougyman

Copy link
Copy Markdown
Member Author

Follow-up fix: callout text attribute refs inside backticks

The prior commit used \:{image-major-tag}`and`:{image-pin-example}` in the callout list item (<2>). AsciiDoc inline code spans suppress attribute substitution, so these would have rendered literally as :{image-major-tag}and:{image-pin-example}rather than expanding to:v1and:v1.13.2`.

Fixed by using literal \:v1`and`:v1.13.2`in the callout text. The source block (inside[source,sh,subs="attributes+"]) is unaffected — attribute expansion works there as designed. The {image-major-tag}and{image-pin-example}` document attributes at the top of the file continue to be tracked by release-please for future releases.

@bougyman
bougyman merged commit 9e5d7e0 into main Aug 18, 2026
2 checks passed
@bougyman
bougyman deleted the CRY-59-sync-readme-container-tags branch August 18, 2026 17:53
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.

2 participants