-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add support for passing new Spring Boot JarLauncher class #178
Add support for passing new Spring Boot JarLauncher class #178
Conversation
Thanks for the contribution! Before we can merge this, we need @hjellek to sign the Salesforce Inc. Contributor License Agreement. |
I have signed the CLA now |
springboot/springboot.bzl
Outdated
@@ -385,6 +385,7 @@ def springboot( | |||
name, | |||
java_library, | |||
boot_app_class, | |||
spring_boot_launcher_class = "org.springframework.boot.loader.JarLauncher", |
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.
nitpick: for consistency, can you rename attr to boot_launcher_class
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.
Of course
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.
One small nitpick, otherwise looks good. Thanks!
I am preparing release 2.3.1 with this change. |
Attempt at fixing #177
Works in our project, and has backward compatability.
Open to suggestions for improvements!