Skip to content

Banners show the release version via git describe - #10

Merged
elirancv merged 2 commits into
mainfrom
fix/banner-and-unblock-docs
Jul 7, 2026
Merged

Banners show the release version via git describe#10
elirancv merged 2 commits into
mainfrom
fix/banner-and-unblock-docs

Conversation

@elirancv

@elirancv elirancv commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Follow-up to #9 (the commit raced the merge): instead of versionless banners, every script (stage0/1/2/3 + wsl-tools) resolves the latest git tag at runtime and shows it — v5.2.0 at a release tag, v5.2.0-N-gsha when ahead (honest dev-build style), silent for ZIP copies without .git. One source of truth (the tag), zero per-release maintenance, can never drift like the old 4.3 / 4.3.3 / 4.4.0 / 1.0 set. AGENTS.md invariant updated to match.

🤖 Generated with Claude Code

elirancv added a commit that referenced this pull request Jul 7, 2026
@elirancv
elirancv force-pushed the fix/banner-and-unblock-docs branch from 716d781 to 73553c3 Compare July 7, 2026 05:40
elirancv and others added 2 commits July 7, 2026 08:40
Instead of removing versions outright, derive them: banners display
the latest git tag (e.g. v5.2.0, or v5.2.0-3-gabc123 when ahead of the
tag) resolved at runtime, and stay silent for ZIP copies without .git.
One source of truth - the tag - so banners can never drift again the
way the hardcoded 4.3 / 4.3.3 / 4.4.0 / 1.0 set did.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
git describe exits 128 in tagless/shallow checkouts (and ZIP copies);
the try/catch handles the error but the exit code leaked - CI's
powershell wrapper ends with `exit $LASTEXITCODE`, failing the step
even though the script succeeded. Caught by the Windows 5.1 CI job.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@elirancv
elirancv merged commit 65d18dc into main Jul 7, 2026
4 checks passed
@elirancv
elirancv deleted the fix/banner-and-unblock-docs branch July 7, 2026 05:47
@elirancv elirancv mentioned this pull request Jul 7, 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