Skip to content

Conversation

@simonz130
Copy link

When the bom instructions feature at https://cloud.google.com/logging/docs/reference/libraries?hl=en
there is a weird part there:

<!-- ...
</dependencies>

Comparing to other libraries in java, it looks like we need to add a line break.

When the bom instructions feature at https://cloud.google.com/logging/docs/reference/libraries?hl=en
there is a weird part there:
```xml
<!-- ...
</dependencies>
```

Comparing to other libraries in java, it looks like we need to add a line break.
@simonz130 simonz130 added type: docs Improvement to the documentation for an API. api: logging Issues related to the googleapis/java-logging API. automerge Merge the pull request once unit tests and other checks pass. labels Aug 11, 2021
@simonz130 simonz130 requested review from chingor13 and minherz August 11, 2021 06:00
@simonz130 simonz130 requested a review from a team as a code owner August 11, 2021 06:00
@simonz130 simonz130 self-assigned this Aug 11, 2021
@simonz130 simonz130 requested a review from a team as a code owner August 11, 2021 06:00
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 11, 2021
@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • samples/snippets/pom.xml

Copy link
Contributor

@minherz minherz left a comment

Choose a reason for hiding this comment

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

Should we add an empty line between lines 57 and 58 too:

</dependency>

<!-- [END_EXCLUDE] -->
</dependencies>

or between 58 and 58:

</dependency>
<!-- [END_EXCLUDE] -->

</dependencies>

?

@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 Aug 11, 2021
@minherz minherz merged commit 9d70b96 into master Sep 26, 2021
@minherz minherz deleted the simonz130-patch-1 branch September 26, 2021 09:19
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 27, 2021
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 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.

3 participants