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
🤖 I have created a release \*beep\* \*boop\*
---
## [3.2.0](https://www.github.com/googleapis/java-logging/compare/v3.1.2...v3.2.0) (2021-09-27)


### Features

* Add a sample snippet for use of the tailLogEntries API ([8115888](https://www.github.com/googleapis/java-logging/commit/81158885911e6c150fa04053fc41f95640329548))
* add support for tailLogEntry API ([5a7845a](https://www.github.com/googleapis/java-logging/commit/5a7845a8c6476ca2d80e6fffdcb2f5f061f6260b))


### Bug Fixes

* add space to fix embedding exclusion in docs ([#615](https://www.github.com/googleapis/java-logging/issues/615)) ([9d70b96](https://www.github.com/googleapis/java-logging/commit/9d70b96914ae6217cbe58ab1af93e1d5dfe67ce1))
* fix samples region tags and test flakiness ([#680](https://www.github.com/googleapis/java-logging/issues/680)) ([53481bd](https://www.github.com/googleapis/java-logging/commit/53481bde1e455037cd8102bc1b404ea2c491c235))


### Dependencies

* update dependency com.google.cloud:google-cloud-shared-dependencies to v2.3.0 ([#681](https://www.github.com/googleapis/java-logging/issues/681)) ([9be4047](https://www.github.com/googleapis/java-logging/commit/9be4047f5631c824e3f07cfc8c21211168bd51d4))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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