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

Update ListRepositoryWorkflowRuns to support all query parameters #1498

Merged
merged 1 commit into from
Apr 20, 2020

Conversation

acburdine
Copy link
Contributor

closes #1497

  • update the method to use the more specific ListWorkflowRunsOptions struct

Let me know if I need to update any of the tests as well - I looked through the existing tests and didn't see any that were specifically testing the unique query params of workflow endpoints.

@googlebot googlebot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Apr 20, 2020
closes google#1497
- update the method to use the more specific ListWorkflowRunsOptions struct
@codecov
Copy link

codecov bot commented Apr 20, 2020

Codecov Report

Merging #1498 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1498   +/-   ##
=======================================
  Coverage   68.19%   68.19%           
=======================================
  Files          94       94           
  Lines        8452     8452           
=======================================
  Hits         5764     5764           
  Misses       1819     1819           
  Partials      869      869           
Impacted Files Coverage Δ
github/actions_workflow_runs.go 57.69% <100.00%> (ø)

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 8ac6f14...9a9247a. Read the comment docs.

Copy link
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.

LGTM.
Note that this is a breaking change, but we now make releases on an as-needed basis.
So the next time we release, we will need to bump the version to v32.

Awaiting second LGTM before merging.

@gmlewis gmlewis requested review from gauntface and wesleimp April 20, 2020 17:57
Copy link
Collaborator

@wesleimp wesleimp left a comment

Choose a reason for hiding this comment

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

LGTM 👌

@gmlewis
Copy link
Collaborator

gmlewis commented Apr 20, 2020

Thank you, @wesleimp!
Merging.

@gmlewis gmlewis merged commit 7623a89 into google:master Apr 20, 2020
@acburdine acburdine deleted the patch-1 branch April 20, 2020 18:31
@gmlewis gmlewis added the Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). label Apr 24, 2020
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
Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). 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.

ListRepositoryWorkflowRuns doesn't support all query parameters
4 participants