Skip to content

Remove legacy oasdiff.yml workflow#16

Merged
reuvenharrison merged 1 commit intomainfrom
fix/remove-legacy-oasdiff-workflow
Apr 9, 2026
Merged

Remove legacy oasdiff.yml workflow#16
reuvenharrison merged 1 commit intomainfrom
fix/remove-legacy-oasdiff-workflow

Conversation

@reuvenharrison
Copy link
Copy Markdown
Collaborator

Summary

  • Removes oasdiff.yml (v0.0.31, git-revision-based) which was overwriting the review links added by pr-comment.yml (v0.0.34)
  • Both workflows used the same <!-- oasdiff-pr-report --> comment anchor, causing the last writer to win — and v0.0.31 doesn't support review links

Test plan

  • Merge this PR
  • Open a new test PR — only pr-comment.yml and multi-file-demo.yml should run
  • Review link should appear correctly in the PR comment

🤖 Generated with Claude Code

The pr-comment.yml (v0.0.34) supersedes oasdiff.yml (v0.0.31). Both
write to the same <!-- oasdiff-pr-report --> comment anchor, so the
old workflow was overwriting the review links added by pr-comment.yml.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@oasdiff
Copy link
Copy Markdown

oasdiff bot commented Apr 9, 2026

oasdiff API Change Report

No API changes detected.


Compared at 8cf20cc · 📋 Review & approve changes

@reuvenharrison reuvenharrison merged commit aec2717 into main Apr 9, 2026
3 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