Skip to content
This repository was archived by the owner on Dec 8, 2021. It is now read-only.

chore: disable warning in test file to simplify code#1264

Merged
devjgm merged 1 commit intogoogleapis:masterfrom
devjgm:pragma
Feb 13, 2020
Merged

chore: disable warning in test file to simplify code#1264
devjgm merged 1 commit intogoogleapis:masterfrom
devjgm:pragma

Conversation

@devjgm
Copy link
Copy Markdown
Contributor

@devjgm devjgm commented Feb 13, 2020

Disables -Wmissing-field-initializers in one unit test file to improve
the readability of our test code. Note: This flag is enabled by -Wextra.
More details in the comment in the code.


This change is Reviewable

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 13, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 13, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@e90e64e). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1264   +/-   ##
=========================================
  Coverage          ?   94.82%           
=========================================
  Files             ?      184           
  Lines             ?    13810           
  Branches          ?        0           
=========================================
  Hits              ?    13096           
  Misses            ?      714           
  Partials          ?        0
Impacted Files Coverage Δ
...gle/cloud/spanner/internal/connection_impl_test.cc 97.93% <100%> (ø)

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 e90e64e...bbce819. Read the comment docs.

Disables -Wmissing-field-initializers in one unit test file to improve
the readability of our test code. Note: This flag is enabled by -Wextra.
More details in the comment in the code.
@devjgm devjgm requested a review from coryan February 13, 2020 17:47
@devjgm devjgm marked this pull request as ready for review February 13, 2020 17:47
Copy link
Copy Markdown
Contributor

@coryan coryan left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@devjgm devjgm merged commit 23b3570 into googleapis:master Feb 13, 2020
@devjgm devjgm deleted the pragma branch February 13, 2020 18:22
devjgm added a commit to devjgm/google-cloud-cpp that referenced this pull request May 7, 2020
…le-cloud-cpp-spanner#1264)

Disables -Wmissing-field-initializers in one unit test file to improve
the readability of our test code. Note: This flag is enabled by -Wextra.
More details in the comment in the code.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants