Skip to content

Publish the generated version on the working-1.6 badge - #41

Merged
mastacontrola merged 1 commit into
mainfrom
fix/beta-version-badge
Sep 10, 2026
Merged

mastacontrola merged 1 commit into
mainfrom
fix/beta-version-badge

Conversation

@mastacontrola

Copy link
Copy Markdown
Member

Problem

The Beta badge in the fogproject README renders "custom badge: invalid properties: message". badges/working-1.6.json carries "message": "".

Cause

working-1.6 carries .githooks/lib/write-version-file.sh, so the sweep skips the compute step there (#39). The badge step still read steps.commit.outputs.final_version || steps.compute.outputs.version. Both are unset on that branch, so it published an empty message. shields.io rejects an empty message.

Fix

  • The badge step runs the branch's own write-version-file.sh, the same generator bin/installfog.sh uses, and reads FOG_VERSION from packages/web/commons/version.php.
  • An empty version fails the step. It no longer publishes a broken badge from a green job.
  • The job summary no longer prints an empty version as "already correct" on those branches.
  • CLAUDE.md describes the new badge source.

Verification

  • Ran the new logic in a --filter=blob:none clone of working-1.6 at e69ef2cc9: 1.6.0-beta.5360. This matches the installer's own awk read of the same file and master..HEAD --count.
  • Dispatched this branch with branch=working-1.6, version_only=true. Result below.

Not in this PR

The sweep has also failed daily on dev-branch since 2026-09-06. The tag archive/feature-fog2-gui is now the newest tag, so fog-version.sh computes archive/feature-fog2-gui.2480 and sed fails on the slash. The fix belongs in fogproject's fog-version.sh and goes in separate PRs there.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RANCjBL6XSavKYf58MZKZj

A branch that carries .githooks/lib/write-version-file.sh skips the
compute step, so the badge step read two unset outputs and wrote an
empty message. shields.io rejects that as "invalid properties: message",
and the Beta badge in the fogproject README has been broken since
2026-08-30.

The badge step now runs the branch's own write-version-file.sh, the
same generator bin/installfog.sh uses, and reads FOG_VERSION back out of
packages/web/commons/version.php. An empty version fails the step
instead of publishing a broken badge. The job summary no longer reports
an empty version as "already correct" on those branches.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RANCjBL6XSavKYf58MZKZj
@mastacontrola

Copy link
Copy Markdown
Member Author

Dispatch from this branch (run 34511435938, branch=working-1.6, version_only=true) succeeded. badges/working-1.6.json now carries "message": "1.6.0-beta.5360".

@mastacontrola
mastacontrola merged commit 7e30588 into main Sep 10, 2026
2 checks passed
@mastacontrola
mastacontrola deleted the fix/beta-version-badge branch September 10, 2026 17:59
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