Skip to content

Auto-publish the DMG to Releases on merge to main + prominent README download link - #19

Merged
missingbulb merged 1 commit into
mainfrom
claude/laugh-detection-logging-lq957s
Jul 13, 2026
Merged

Auto-publish the DMG to Releases on merge to main + prominent README download link#19
missingbulb merged 1 commit into
mainfrom
claude/laugh-detection-logging-lq957s

Conversation

@missingbulb

Copy link
Copy Markdown
Owner

Fixes the empty Releases area and the 404ing latest-download link.

Why it was broken: release-macos-dmg.yml only fired on a v* tag push (blocked from the Claude Code session's git proxy, 403) or a manual UI publish — so no release ever got created.

Fix:

  • release-macos-dmg.yml now also runs on push to main (mac/** paths): it derives v<version> from Info.plist and creates/updates that GitHub Release with LaughCounter.dmg attached, via GITHUB_TOKEN (which can tag/release from Actions). release: published + workflow_dispatch triggers remain. A release made with GITHUB_TOKEN doesn't re-fire the release event, so no loop.
  • build-macos-dmg.yml now excludes main (the release workflow owns it) to avoid a duplicate build.
  • Root README gets a prominent top-of-file Download line (latest DMG + all-releases link); mac/README documents the auto-publish-on-merge flow.

Once this merges, the push to main will publish the v0.2.1 release automatically and the latest-download link will resolve.

Closes #18

🤖 Generated with Claude Code


Generated by Claude Code

…ad link

Releases never appeared: release-macos-dmg.yml only fired on a v* tag push
(blocked from the Claude Code session's git proxy) or a manual UI publish, so the
Release area stayed empty and releases/latest/download/LaughCounter.dmg 404'd.

- release-macos-dmg.yml now also runs on push to main (mac/** paths): it derives
  v<version> from Info.plist and creates/updates that GitHub Release with the DMG
  via GITHUB_TOKEN (which can tag/release from Actions, unlike the session). The
  release: published and workflow_dispatch triggers remain.
- build-macos-dmg.yml excludes main now (release-macos-dmg owns main) to avoid a
  duplicate build.
- Root README gets a prominent top-of-file download line (latest DMG + all
  releases); mac/README documents the auto-publish-on-merge flow.

Closes #18

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ATTAHuLKSR8K7ZFawgbncJ
@missingbulb
missingbulb merged commit 4f9501e into main Jul 13, 2026
1 check passed
missingbulb added a commit that referenced this pull request Jul 14, 2026
The two release-automation gotchas learned wiring release-macos-dmg.yml were
only recorded in commit messages (#13, #19) and the workflow YAML comments, so a
future session touching the release flow wouldn't be pointed at them. Re-home
them into a local instruction doc reachable from CLAUDE.md:

- A Claude Code web session's git proxy can't push v* tags, so a tag-triggered
  release never fires from a session — trigger on push-to-main instead (#18/#19).
- A UI-published Release doesn't emit push: tags — add release: published so a
  hand-published Release still builds/attaches the DMG (#11/#13).

Creates the dev/procedures/ instruction-doc scaffold (none existed) and imports
it from the root CLAUDE.md.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018DKJNecs661QFXT5YZEeTn
@missingbulb
missingbulb deleted the claude/laugh-detection-logging-lq957s branch July 23, 2026 11:55
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.

Releases never appear in the Release area; make CI auto-publish + prominent README download link

2 participants