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

refactor: copy IsProtoEqual from spanner #179

Merged
merged 1 commit into from
Feb 14, 2020
Merged

refactor: copy IsProtoEqual from spanner #179

merged 1 commit into from
Feb 14, 2020

Conversation

coryan
Copy link
Contributor

@coryan coryan commented Feb 14, 2020

Create a new library to host common testing utilities that depend on
Protobuf, as the GCS library does not depend on protobuf and we do not
want to depend on protobuf for its tests.


This change is Reviewable

Create a new library to host common testing utilities that depend on
Protobuf, as the GCS library does not depend on protobuf and we do not
want to depend on protobuf for its tests.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 14, 2020
@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #179 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
+ Coverage   96.22%   96.23%   +0.01%     
==========================================
  Files          94       97       +3     
  Lines        4395     4411      +16     
==========================================
+ Hits         4229     4245      +16     
  Misses        166      166
Impacted Files Coverage Δ
google/cloud/testing_util/is_proto_equal.cc 100% <100%> (ø)
google/cloud/testing_util/is_proto_equal_test.cc 100% <100%> (ø)
google/cloud/testing_util/is_proto_equal.h 100% <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 80fcf05...4557c26. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #179 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
+ Coverage   96.22%   96.23%   +0.01%     
==========================================
  Files          94       97       +3     
  Lines        4395     4411      +16     
==========================================
+ Hits         4229     4245      +16     
  Misses        166      166
Impacted Files Coverage Δ
google/cloud/testing_util/is_proto_equal.cc 100% <100%> (ø)
google/cloud/testing_util/is_proto_equal_test.cc 100% <100%> (ø)
google/cloud/testing_util/is_proto_equal.h 100% <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 80fcf05...4557c26. Read the comment docs.

@coryan coryan marked this pull request as ready for review February 14, 2020 21:50
@coryan coryan merged commit c1a0fa6 into googleapis:master Feb 14, 2020
@coryan coryan deleted the refactor-is-proto-equal branch February 15, 2020 00:00
coryan added a commit to coryan/google-cloud-cpp that referenced this pull request Apr 24, 2020
…-common#179)

Create a new library to host common testing utilities that depend on
Protobuf, as the GCS library does not depend on protobuf and we do not
want to depend on protobuf for its tests.
coryan added a commit to coryan/google-cloud-cpp that referenced this pull request Apr 24, 2020
…-common#179)

Create a new library to host common testing utilities that depend on
Protobuf, as the GCS library does not depend on protobuf and we do not
want to depend on protobuf for its tests.
coryan added a commit to coryan/google-cloud-cpp that referenced this pull request Apr 24, 2020
…-common#179)

Create a new library to host common testing utilities that depend on
Protobuf, as the GCS library does not depend on protobuf and we do not
want to depend on protobuf for its tests.
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