Skip to content

Conversation

@simonz130
Copy link

@simonz130 simonz130 commented May 18, 2021

  • Appropriate docs were updated (if necessary)

Fixes #393 ☕️

@simonz130 simonz130 added the api: logging Issues related to the googleapis/java-logging-logback API. label May 18, 2021
@simonz130 simonz130 requested a review from a team as a code owner May 18, 2021 07:45
@simonz130 simonz130 self-assigned this May 18, 2021
@simonz130 simonz130 requested a review from a team as a code owner May 18, 2021 07:45
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 18, 2021
@simonz130 simonz130 added automerge Merge the pull request once unit tests and other checks pass. type: docs Improvement to the documentation for an API. kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels May 18, 2021
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels May 18, 2021
@gcf-merge-on-green
Copy link

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label May 18, 2021
README.md Outdated
Java is a registered trademark of Oracle and/or its affiliates.

[product-docs]: https://cloud.google.com/logging-logback/docs/
[product-docs]: https://cloud.google.com/logging/setup/java/
Copy link
Contributor

Choose a reason for hiding this comment

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

This appears to be a 404. Should it be https://cloud.google.com/logging/docs/setup/java?

Copy link
Author

Choose a reason for hiding this comment

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

updated

@freelerobot freelerobot dismissed daniel-sanche’s stale review May 19, 2021 01:18

Updated by Simon

@freelerobot freelerobot self-requested a review May 19, 2021 01:18
Copy link
Contributor

@freelerobot freelerobot left a comment

Choose a reason for hiding this comment

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

LGTM

I dont think this file will be overwritten given :

"generatedFiles": [

@simonz130 simonz130 merged commit 2ff2886 into master May 19, 2021
@simonz130 simonz130 deleted the fixreadme branch May 19, 2021 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging Issues related to the googleapis/java-logging-logback API. cla: yes This human has signed the Contributor License Agreement. type: docs Improvement to the documentation for an API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken links in README : https://cloud.google.com/logging-logback/docs/

4 participants