Skip to content

Enforce README version badge == DIX_VERSION in CI; add bump-version.sh - #25

Merged
CryptoJones merged 1 commit into
mainfrom
add-version-badge-check
Jul 5, 2026
Merged

Enforce README version badge == DIX_VERSION in CI; add bump-version.sh#25
CryptoJones merged 1 commit into
mainfrom
add-version-badge-check

Conversation

@CryptoJones

Copy link
Copy Markdown
Owner

Folds the version-badge bump into the release step so the static badge can't drift.

  • ci.yml gains a version-badge job: extracts DIX_VERSION (from waveform.c) and the README badge/version-… value, and fails the build if they differ.
  • scripts/bump-version.sh <version> updates both the DIX_VERSION macro and the README badge in one shot (BSD/GNU-sed portable).

Net: releasing = scripts/bump-version.sh X.Y.Z, and CI goes red if the badge is ever out of sync. Verified the check passes at 3.5.4 and the script is idempotent.

🤖 Generated with Claude Code

Proudly Made in Nebraska. Go Big Red! 🌽 https://xkcd.com/2347/

Makes the static README version badge part of the release step so it can't drift
from the shipped version:
- ci.yml gains a `version-badge` job that extracts DIX_VERSION (waveform.c) and
  the README `badge/version-…` value and fails the build if they differ.
- scripts/bump-version.sh <version> updates both the DIX_VERSION macro and the
  README badge in one shot (BSD/GNU-sed portable), so releasing = one command +
  CI red if you forget.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JpVzsy8Wrc9DYEXns2Njzb
@CryptoJones
CryptoJones merged commit 1266c11 into main Jul 5, 2026
@CryptoJones
CryptoJones deleted the add-version-badge-check branch July 5, 2026 15:31
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