Skip to content

Remove required constraint from name property#15

Open
reuvenharrison wants to merge 2 commits intomainfrom
test-review-page
Open

Remove required constraint from name property#15
reuvenharrison wants to merge 2 commits intomainfrom
test-review-page

Conversation

@reuvenharrison
Copy link
Copy Markdown
Collaborator

Summary

  • Removes required constraint from name property in GroupView schema
  • This is a breaking change to test the review page end-to-end

Test plan

  • PR comment is posted by the workflow with a review link
  • Review page shows the breaking change with correct labels (filename only, not full URL)
  • Base uses the base commit SHA (immutable), not the branch name

🤖 Generated with Claude Code

@oasdiff
Copy link
Copy Markdown

oasdiff bot commented Apr 9, 2026

oasdiff API Change Report

🟢 2 info

Severity Change Path Review
🟢 the response property 'tag' became required for the status '200' GET /pets View
🟢 the response property 'tag' became required for the status '200' GET /pets/{petId} View

Compared at 6ed7a0e · 📋 Review & approve changes

@oasdiff
Copy link
Copy Markdown

oasdiff bot commented Apr 9, 2026

oasdiff API Change Report

No API changes detected.


Compared at 8ed186c · 📋 Review & approve changes

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