Skip to content

test: GitHub App — add required email field#10

Closed
reuvenharrison wants to merge 1 commit intomainfrom
test
Closed

test: GitHub App — add required email field#10
reuvenharrison wants to merge 1 commit intomainfrom
test

Conversation

@reuvenharrison
Copy link
Collaborator

Testing that the GitHub App installation token is used for the PR comment and commit status — both on initial posting and after a reviewer approves the breaking change.

@oasdiff
Copy link

oasdiff bot commented Mar 22, 2026

oasdiff API Change Report

🔴 1 breaking change · 🟢 2 info

Severity Change Path Review
🔴 added the new required request property 'data/email' POST /api/v1.0/groups Approved by @reuven
🟢 added the required property 'data/email' to the response with the '400' status POST /api/v1.0/groups View
🟢 added the required property 'data/email' to the response with the '200' status POST /api/v1.0/groups View

Compared at 5868823 · 📋 Review & approve changes

@reuvenharrison
Copy link
Collaborator Author

Test complete — GitHub App installation token working correctly. Commit status flipped from pending → success after approval with no 401 errors.

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