Skip to content

Conversation

@chingor13
Copy link
Contributor

Custom content is now in .readme-partials.yaml so any updates need to go there for now. This will allow automatic updates to the install sections for the latest libraries-bom versions and keep our README format in sync with the other client libraries.

Also fixes a few broken links to the google-cloud-examples that still live in the monorepo.

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

codecov bot commented Apr 13, 2020

Codecov Report

Merging #238 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #238   +/-   ##
=========================================
  Coverage     63.52%   63.52%           
  Complexity      540      540           
=========================================
  Files            30       30           
  Lines          4776     4776           
  Branches        431      431           
=========================================
  Hits           3034     3034           
  Misses         1580     1580           
  Partials        162      162           

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 b9d0a70...45c0d84. Read the comment docs.

Copy link
Contributor

@frankyn frankyn left a comment

Choose a reason for hiding this comment

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

Overall, LGTM, pending one question.

README.md Outdated
Java 7 or above is required for using this client.

Contributions to this library are always welcome and highly encouraged.
## Versioning
Copy link
Contributor

Choose a reason for hiding this comment

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

@chingor13 how can I update versioning information for this library based on discussions around the Storage interface?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll need to add an opt-out in the shared template configuration

Copy link
Contributor

Choose a reason for hiding this comment

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

Gotcha, is this something you want to do after this PR?

@frankyn frankyn added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 14, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 14, 2020
@frankyn frankyn merged commit ebab7e6 into master Apr 14, 2020
@frankyn frankyn deleted the generate-readme branch April 14, 2020 21:17
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.

4 participants