Skip to content

Conversation

@devjgm
Copy link
Contributor

@devjgm devjgm commented Apr 14, 2021

Also fixed an issue where the pubsub protos were missing a needed dep.

This PR refactors the logic to run the CMake and Makefile quickstart
builds into a library, because it turns out that this is a useful way to
verify that our installed artifacts work, and we end up calling this
from a few places.


This change is Reviewable

Also fixed an issue where the pubsub protos were missing a needed dep.

This PR refactors the logic to run the CMake and Makefile quickstart
builds into a library, because it turns out that this is a useful way to
verify that our installed artifacts work, and we end up calling this
from a few places.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 14, 2021
@codecov
Copy link

codecov bot commented Apr 14, 2021

Codecov Report

Merging #6257 (62b717c) into master (fb33e9b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6257   +/-   ##
=======================================
  Coverage   95.68%   95.68%           
=======================================
  Files        1178     1178           
  Lines      105216   105216           
=======================================
  Hits       100671   100671           
  Misses       4545     4545           

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 fb33e9b...62b717c. Read the comment docs.

@devjgm devjgm marked this pull request as ready for review April 14, 2021 21:34
@devjgm devjgm requested a review from a team as a code owner April 14, 2021 21:34
Copy link
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.

Nice refactoring.

@devjgm devjgm merged commit ffc2413 into googleapis:master Apr 14, 2021
@devjgm devjgm deleted the gcb-shared branch April 14, 2021 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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