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.

Sign up for GitHub

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

Jump to bottom

Mark a single line of Vision uncovered. #2169

Merged
merged 1 commit into from
Mar 31, 2017
Merged

Mark a single line of Vision uncovered. #2169

merged 1 commit into from
Mar 31, 2017

Conversation

lukesneeringer
Copy link
Contributor

I am declaring bankruptcy on the last line of coverage for now. I just want coveralls to stop giving Xes. I will go back and refactor that code later (it will be a non-trivial effort).

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 31, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 100.0% when pulling 565ab9c on lukesneeringer:coverage-bankruptcy into 0b2c4c3 on GoogleCloudPlatform:master.

@stephenplusplus
Copy link
Contributor

OMG!

@stephenplusplus stephenplusplus merged commit 86ac806 into googleapis:master Mar 31, 2017
@lukesneeringer lukesneeringer deleted the coverage-bankruptcy branch March 31, 2017 21:55
@sofisl sofisl mentioned this pull request Jan 17, 2023
Merged
4 tasks
sofisl pushed a commit that referenced this pull request Jan 17, 2023
@lukesneeringer @stephenplusplus
Mark a single line of Vision uncovered. (#2169)
0aad6b9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephenplusplus stephenplusplus Awaiting requested review from stephenplusplus

Assignees
No one assigned
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lukesneeringer @coveralls @stephenplusplus @googlebot