Skip to content

doc: fix cmake instructions in top-level README#7057

Merged
coryan merged 1 commit intogoogleapis:mainfrom
VPeruS:readme-correct-cmake
Jul 26, 2021
Merged

doc: fix cmake instructions in top-level README#7057
coryan merged 1 commit intogoogleapis:mainfrom
VPeruS:readme-correct-cmake

Conversation

@VPeruS
Copy link
Copy Markdown
Contributor

@VPeruS VPeruS commented Jul 25, 2021

Reordered command line options, "-C" option is part of the git's general options.


This change is Reviewable

@VPeruS VPeruS requested a review from a team July 25, 2021 12:01
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 25, 2021
@coryan
Copy link
Copy Markdown
Contributor

coryan commented Jul 25, 2021

Thanks for the pull request. Do you mind also the file that generates this README?

git clone -C $HOME https://github.com/microsoft/vcpkg.git

The CI builds will not pass until that is fixed. If this is too annoying we would be happy to make the changes ourselves.

Reordered command line options, "-C" option is part of the git's general options.
@VPeruS VPeruS force-pushed the readme-correct-cmake branch from 657ea82 to 4ff595b Compare July 26, 2021 16:44
@VPeruS
Copy link
Copy Markdown
Contributor Author

VPeruS commented Jul 26, 2021

Updated bash script.

@coryan
Copy link
Copy Markdown
Contributor

coryan commented Jul 26, 2021

/gcbrun

@coryan coryan added the kokoro:run Add this label to force Kokoro to re-run the tests. label Jul 26, 2021
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Jul 26, 2021
@coryan coryan changed the title [readme] Fixed cmake instructions doc: fixed cmake instructions Jul 26, 2021
@coryan coryan changed the title doc: fixed cmake instructions doc: fix cmake instructions in top-level README Jul 26, 2021
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 4ff595b94d579f66c37c98fc4c80640757ab434d

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 26, 2021

Codecov Report

Merging #7057 (4ff595b) into main (727bc4f) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7057   +/-   ##
=======================================
  Coverage   94.48%   94.48%           
=======================================
  Files        1304     1304           
  Lines      112319   112319           
=======================================
+ Hits       106124   106126    +2     
+ Misses       6195     6193    -2     
Impacted Files Coverage Δ
google/cloud/pubsub/subscriber_connection_test.cc 97.20% <0.00%> (-0.70%) ⬇️
google/cloud/pubsub/samples/samples.cc 91.75% <0.00%> (+0.07%) ⬆️
google/cloud/grpc_error_delegate.cc 100.00% <0.00%> (+4.16%) ⬆️

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 727bc4f...4ff595b. Read the comment docs.

@coryan coryan merged commit 04b9b79 into googleapis:main Jul 26, 2021
@coryan
Copy link
Copy Markdown
Contributor

coryan commented Jul 26, 2021

Thanks again for the PR.

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.

5 participants