Skip to content

Fix dropped test error - #2858

Merged
gmlewis merged 1 commit into
google:masterfrom
alrs:test-err
Aug 7, 2023
Merged

Fix dropped test error#2858
gmlewis merged 1 commit into
google:masterfrom
alrs:test-err

Conversation

@alrs

@alrsalrs commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

This picks up a dropped err variable in tests and clarifies the error message.

@gmlewisgmlewis changed the title github: fix dropped test errorFix dropped test errorAug 7, 2023
@codecov

codecovBot commented Aug 7, 2023

Copy link
Copy Markdown

Codecov Report

Merging #2858 (b5fd0f5) into master (a8da03b) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #2858 +/- ##
=======================================
Coverage 98.06% 98.06% =======================================
Files 138 138 Lines 12327 12327 =======================================
Hits 12089 12089 Misses 162 162 Partials 76 76 

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @alrs !
LGTM.
Merging.

@gmlewis
gmlewis merged commit d24bbc2 into google:masterAug 7, 2023
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@alrs@gmlewis