Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deserialize requested_action field on check_run events. #2197

Merged

Conversation

mitchdenny
Copy link
Contributor

@mitchdenny mitchdenny commented Jun 2, 2020

This PR closes #2195. It adds support for handling the requested_action field when present on Check Run event payloads. This field is documented in the developer docs, but wasn't exposed in Octokit for .NET:

https://developer.github.com/webhooks/event-payloads/#check_run

  • New type defined
  • Type exposed on existing check run event payload
  • Test JSON updated and assertion added

@codecov
Copy link

codecov bot commented Jun 2, 2020

Codecov Report

Merging #2197 into master will decrease coverage by 0.84%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #2197      +/-   ##
==========================================
- Coverage   65.88%   65.03%   -0.85%     
==========================================
  Files         546      547       +1     
  Lines       14293    14278      -15     
  Branches      838      835       -3     
==========================================
- Hits         9417     9286     -131     
  Misses       4718     4718              
- Partials      158      274     +116     
Impacted Files Coverage Δ
Octokit/Models/Response/CheckRunRequestedAction.cs 42.85% <42.85%> (ø)
.../Response/ActivityPayloads/CheckRunEventPayload.cs 100.00% <100.00%> (ø)
Octokit/Helpers/AuthorizationExtensions.cs 82.35% <0.00%> (-17.65%) ⬇️
Octokit/Clients/MigrationsClient.cs 83.33% <0.00%> (-16.67%) ⬇️
Octokit/Clients/StatisticsClient.cs 86.20% <0.00%> (-13.80%) ⬇️
Octokit/Clients/GitHubAppInstallationsClient.cs 50.00% <0.00%> (-12.50%) ⬇️
Octokit/Clients/RepositoryInvitationsClient.cs 67.85% <0.00%> (-10.72%) ⬇️
Octokit/Clients/ApiPagination.cs 40.00% <0.00%> (-10.00%) ⬇️
Octokit/Clients/ProjectColumnsClient.cs 73.91% <0.00%> (-8.70%) ⬇️
Octokit/Clients/Enterprise/EnterpriseLdapClient.cs 66.66% <0.00%> (-8.34%) ⬇️
... and 31 more

@mitchdenny
Copy link
Contributor Author

@shiftkey what is the process to get this reviewed?

@shiftkey
Copy link
Member

shiftkey commented Jun 5, 2020

@mitchdenny a bit of patience - aiming to throw some spare time on the weekend towards these side projects as Life™ and Work™ have been hectic this past week.

Copy link
Member

@shiftkey shiftkey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @mitchdenny!

I'm aiming to cut a release over the weekend to test out some infrastructure changes, so I'll ensure this goes out with that...

@shiftkey shiftkey merged commit b9d1f44 into octokit:master Jun 5, 2020
@mitchdenny
Copy link
Contributor Author

Thbaks @shiftkey

@shiftkey
Copy link
Member

shiftkey commented Jun 7, 2020

release_notes: Deserialize requested_action field on check_run events

@nickfloyd nickfloyd added Type: Bug Something isn't working as documented and removed category: bug labels Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working as documented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CheckRun does not contain requested_action field
3 participants