Skip to content

CheckSuiteEvent and CheckRunEvent: fix doc typos#1168

Merged
gmlewis merged 1 commit intogoogle:masterfrom
evanelias:fix-checks-actions-doc
May 8, 2019
Merged

CheckSuiteEvent and CheckRunEvent: fix doc typos#1168
gmlewis merged 1 commit intogoogle:masterfrom
evanelias:fix-checks-actions-doc

Conversation

@evanelias
Copy link
Copy Markdown
Contributor

Just fixing minor godoc typos -- the checks API uses "rerequested" (no dash) rather than "re-requested", per GitHub API docs:

Also confirmed via actual interactions with the GitHub API; when a check suite or check run is rerequested, no dash was present in the action string of the events received.

Thanks!

The checks API uses "rerequested" (no dash) rather than "re-requested", as
per GitHub API docs:

* https://developer.github.com/v3/activity/events/types/#checksuiteevent
* https://developer.github.com/v3/activity/events/types/#checkrunevent

Also confirmed via actual interactions with the GitHub API -- no dash was
present in the action of the events received when a check suite or run is
rerequested.
@googlebot googlebot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label May 8, 2019
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2019

Codecov Report

Merging #1168 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1168   +/-   ##
======================================
  Coverage    70.2%   70.2%           
======================================
  Files          84      84           
  Lines        5830    5830           
======================================
  Hits         4093    4093           
  Misses        952     952           
  Partials      785     785
Impacted Files Coverage Δ
github/event_types.go 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fef44b...973e9b5. Read the comment docs.

Copy link
Copy Markdown
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @evanelias!
LGTM.
Merging.

@gmlewis gmlewis merged commit f2672b1 into google:master May 8, 2019
n1lesh pushed a commit to n1lesh/go-github that referenced this pull request Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indication that the PR author has signed a Google Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants