Skip to content

Fix release checksum verification path - #5

Merged
okturan merged 1 commit into
masterfrom
codex/fix-macnetstats-release-checksum
Jul 18, 2026
Merged

Fix release checksum verification path#5
okturan merged 1 commit into
masterfrom
codex/fix-macnetstats-release-checksum

Conversation

@okturan

@okturan okturan commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Cause

The first v1 tag run built dist/macnetstats.sha256, then verified that manifest from the repository root. Because the manifest intentionally contains the asset basename, sha256sum looked for ./macnetstats instead of dist/macnetstats.

Fix

  • Verify the checksum from inside the artifact directory.
  • Add the full package/checksum/version dry run to the pull-request verifier, so path or packaging regressions fail before another tag is created.

Local verification

  • Release candidate assembled in a clean temporary directory.
  • SHA-256 verification passed.
  • Packaged script reported MacNetStatsTerm 1.0.0.
  • Workflow YAML parsed.
  • git diff --check passed.

No GitHub Release was created by the failed tag run.

@okturan
okturan merged commit a1c9e20 into master Jul 18, 2026
4 checks passed
@okturan
okturan deleted the codex/fix-macnetstats-release-checksum branch July 18, 2026 20:38
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