-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: add space to fix embedding exclusion in docs #615
Conversation
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.
Warning: This pull request is touching the following templated files:
|
There was a problem hiding this 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>
?
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. |
🤖 I have created a release \*beep\* \*boop\* --- ## [3.2.0](https://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://github.com/googleapis/java-logging/commit/81158885911e6c150fa04053fc41f95640329548)) * add support for tailLogEntry API ([5a7845a](https://github.com/googleapis/java-logging/commit/5a7845a8c6476ca2d80e6fffdcb2f5f061f6260b)) ### Bug Fixes * add space to fix embedding exclusion in docs ([#615](https://github.com/googleapis/java-logging/issues/615)) ([9d70b96](https://github.com/googleapis/java-logging/commit/9d70b96914ae6217cbe58ab1af93e1d5dfe67ce1)) * fix samples region tags and test flakiness ([#680](https://github.com/googleapis/java-logging/issues/680)) ([53481bd](https://github.com/googleapis/java-logging/commit/53481bde1e455037cd8102bc1b404ea2c491c235)) ### Dependencies * update dependency com.google.cloud:google-cloud-shared-dependencies to v2.3.0 ([#681](https://github.com/googleapis/java-logging/issues/681)) ([9be4047](https://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).
When the bom instructions feature at https://cloud.google.com/logging/docs/reference/libraries?hl=en
there is a weird part there:
Comparing to other libraries in java, it looks like we need to add a line break.