Skip to content

feat(changelog): emit free review link when changes are found#103

Merged
reuvenharrison merged 2 commits intomainfrom
feat/changelog-action-review-link
Apr 26, 2026
Merged

feat(changelog): emit free review link when changes are found#103
reuvenharrison merged 2 commits intomainfrom
feat/changelog-action-review-link

Conversation

@reuvenharrison
Copy link
Copy Markdown
Contributor

Summary

Mirrors the existing pattern in the breaking action: when oasdiff changelog finds changes, emit a ::notice:: annotation and a $GITHUB_STEP_SUMMARY markdown link pointing at the free review page at oasdiff.com.

The wording matches the breaking action (Review & approve these API changes) because the changelog action surfaces all severities, and warn/err changes still need approval to clear the merge gate just like breaking ones.

Test plan

  • Run the action on a PR that introduces an API change and confirm a ::notice:: annotation appears with a link to https://www.oasdiff.com/review?....
  • Confirm the same link is rendered as a clickable markdown header in the GitHub Actions step summary.
  • Run the action on a PR with no API changes and confirm no notice or step summary entry is emitted.
  • Click the link and confirm it loads the free review page with the expected base/revision files.

🤖 Generated with Claude Code

reuvenharrison and others added 2 commits April 26, 2026 19:41
Mirrors the breaking action: when oasdiff changelog finds changes, emit
a ::notice:: annotation and a $GITHUB_STEP_SUMMARY markdown link
pointing at the free review page on oasdiff.com.

Wording matches the breaking action ("Review & approve these API
changes") because changelog surfaces all severities, and warn/err
changes still gate the merge and need approval just like breaking ones.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The new review-link emission uses jq to read the PR head SHA from
GITHUB_EVENT_PATH and to URL-encode path arguments. The breaking and
pr-comment Dockerfiles already install it; mirror that here.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@reuvenharrison reuvenharrison merged commit c002f99 into main Apr 26, 2026
24 checks passed
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