Skip to content

Fix set check suite preferences options#1156

Merged
gmlewis merged 1 commit into
google:masterfrom
ardakuyumcu:set-check-suite-preferences-opts
Apr 18, 2019
Merged

Fix set check suite preferences options#1156
gmlewis merged 1 commit into
google:masterfrom
ardakuyumcu:set-check-suite-preferences-opts

Conversation

@ardakuyumcu
Copy link
Copy Markdown
Contributor

Fixes #1155

@googlebot googlebot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Apr 17, 2019
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 17, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1156   +/-   ##
=======================================
  Coverage   70.17%   70.17%           
=======================================
  Files          84       84           
  Lines        5824     5824           
=======================================
  Hits         4087     4087           
  Misses        952      952           
  Partials      785      785
Impacted Files Coverage Δ
github/checks.go 59.7% <ø> (ø) ⬆️

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 a8383c7...7e7664d. 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, @ardakuyumcu!

LGTM.

Awaiting second LGTM before merging.

Note to self: this is a breaking API change so the version needs to be bumped and a new release created after merging.

@gmlewis gmlewis requested a review from gauntface April 17, 2019 01:50
Copy link
Copy Markdown
Contributor

@gauntface gauntface left a comment

Choose a reason for hiding this comment

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

Looks great to me, thanks for the changing & tests :)

@ardakuyumcu
Copy link
Copy Markdown
Contributor Author

Thank you for the swift reviews!

@gmlewis
Copy link
Copy Markdown
Collaborator

gmlewis commented Apr 18, 2019

Thank you, @gauntface!
Merging and then bumping versions.

@gmlewis gmlewis merged commit a6b4602 into google:master Apr 18, 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.

SetCheckSuitePreferences returns 422

4 participants