Skip to content

docs + fix(ci): v1.8.0 docs and release PR status reporting#37

Merged
GraysonCAdams merged 2 commits intomainfrom
docs/v1.8.0-and-ci-fix
Mar 2, 2026
Merged

docs + fix(ci): v1.8.0 docs and release PR status reporting#37
GraysonCAdams merged 2 commits intomainfrom
docs/v1.8.0-and-ci-fix

Conversation

@GraysonCAdams
Copy link
Copy Markdown
Collaborator

Summary

  • Update docs/api.md with enriched GET /api/clips/[id] endpoint and one-reaction-per-user enforcement
  • Update docs/architecture.md directory tree with new ClipOverlay.svelte component
  • Fix release PR check reporting: remove broken close/reopen job from release.yml, add commit status posting to release-pr-checks.yml via GitHub Status API

Context

The close/reopen approach in release.yml never worked because GITHUB_TOKEN events can't trigger other workflows (GitHub limitation). The new approach uses workflow_run (which always fires) and posts commit statuses directly to the PR head SHA so branch protection sees the results.

Test plan

  • CI passes (lint, format, type-check, tests, build, CodeQL)
  • Next release PR should automatically get check statuses without manual intervention

Remove the unreliable close/reopen trigger-pr-checks job from release.yml.
GITHUB_TOKEN events don't trigger other workflows, so close/reopen never
worked. Instead, release-pr-checks.yml now posts commit statuses directly
to the PR head SHA via the GitHub Status API after workflow_run completes.
@GraysonCAdams GraysonCAdams merged commit b176c47 into main Mar 2, 2026
16 checks passed
@GraysonCAdams GraysonCAdams deleted the docs/v1.8.0-and-ci-fix branch March 2, 2026 07:43
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