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

Support options for GetCodeownersErrors #2953

Merged
merged 3 commits into from Oct 10, 2023
Merged

Support options for GetCodeownersErrors #2953

merged 3 commits into from Oct 10, 2023

Conversation

lachlancooper
Copy link
Contributor

@lachlancooper lachlancooper commented Oct 9, 2023

Fixes #2952

@google-cla
Copy link

google-cla bot commented Oct 9, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@lachlancooper lachlancooper marked this pull request as ready for review October 9, 2023 04:01
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #2953 (0b6bb05) into master (74db58f) will increase coverage by 0.00%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

❗ Current head 0b6bb05 differs from pull request most recent head 456ef85. Consider uploading reports for the commit 456ef85 to get more accurate results

@@           Coverage Diff           @@
##           master    #2953   +/-   ##
=======================================
  Coverage   98.17%   98.17%           
=======================================
  Files         145      145           
  Lines       12767    12771    +4     
=======================================
+ Hits        12534    12538    +4     
  Misses        158      158           
  Partials       75       75           
Files Coverage Δ
github/repos_codeowners.go 100.00% <100.00%> (ø)

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.

Thank you, @lachlancooper !

github/repos_codeowners_test.go Show resolved Hide resolved
@gmlewis gmlewis added NeedsReview PR is awaiting a review before merging. Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). labels Oct 9, 2023
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.

Thank you, @lachlancooper !
LGTM.

Awaiting second LGTM+Approval from any other contributor to this repo before merging.

@exageraldo
Copy link
Contributor

LGTM 👍🏻

@gmlewis
Copy link
Collaborator

gmlewis commented Oct 10, 2023

Thank you, @exageraldo !
Merging.

@gmlewis gmlewis removed the NeedsReview PR is awaiting a review before merging. label Oct 10, 2023
@gmlewis gmlewis merged commit fb8f20f into google:master Oct 10, 2023
5 checks passed
@lachlancooper lachlancooper deleted the add-codeowners-options branch October 17, 2023 02:50
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).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Supporting ref parameter for GetCodeownersErrors
3 participants