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

chore: adds spanner-integration-tests labels#1490

Merged
devjgm merged 1 commit intogoogleapis:masterfrom
devjgm:build-tags
May 6, 2020
Merged

chore: adds spanner-integration-tests labels#1490
devjgm merged 1 commit intogoogleapis:masterfrom
devjgm:build-tags

Conversation

@devjgm
Copy link
Copy Markdown
Contributor

@devjgm devjgm commented May 5, 2020

This label will be used in -cpp. This PR also removes a loop in cmake
that appears to be dead code.


This change is Reviewable

This label will be used in `-cpp`. This PR also removes a loop in cmake
that appears to be dead code.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 5, 2020
@devjgm devjgm marked this pull request as ready for review May 5, 2020 21:39
@codecov
Copy link
Copy Markdown

codecov bot commented May 5, 2020

Codecov Report

Merging #1490 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1490      +/-   ##
==========================================
+ Coverage   95.57%   95.59%   +0.01%     
==========================================
  Files         194      192       -2     
  Lines       15925    15924       -1     
==========================================
+ Hits        15220    15222       +2     
+ Misses        705      702       -3     
Impacted Files Coverage Δ
google/cloud/spanner/internal/polling_loop.h 83.33% <0.00%> (-9.53%) ⬇️
google/cloud/spanner/results.h 95.83% <0.00%> (-4.17%) ⬇️
google/cloud/spanner/samples/samples.cc 90.01% <0.00%> (-0.20%) ⬇️
.../spanner/benchmarks/multiple_rows_cpu_benchmark.cc 91.96% <0.00%> (-0.01%) ⬇️
google/cloud/spanner/internal/session.h 100.00% <0.00%> (ø)
google/cloud/spanner/session_pool_options.h 100.00% <0.00%> (ø)
google/cloud/spanner/internal/transaction_impl.h 100.00% <0.00%> (ø)
...cloud/spanner/internal/partial_result_set_resume.h 100.00% <0.00%> (ø)
...oogle/cloud/spanner/internal/database_admin_stub.h
google/cloud/spanner/database_admin_connection.h
... and 10 more

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 74824a4...edb164a. Read the comment docs.

@devjgm devjgm requested review from coryan, devbww, mr-salty and scotthart May 5, 2020 21:40
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 6 of 6 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @devbww, @mr-salty, and @scotthart)

@devjgm devjgm merged commit 4bc4c1a into googleapis:master May 6, 2020
@devjgm devjgm deleted the build-tags branch May 6, 2020 00:12
devjgm added a commit to devjgm/google-cloud-cpp that referenced this pull request May 7, 2020
…-cpp-spanner#1490)

This label will be used in `-cpp`. This PR also removes a loop in cmake
that appears to be dead 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