FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

#142: add Automatic-Module-Name during build by SingingBush · Pull Request #469 · aws/aws-lambda-java-libs · GitHub

#142: add Automatic-Module-Name during build - #469

Open
SingingBush wants to merge 2 commits into
aws:mainfrom
SingingBush:feature/JPMS_automatic_module_name
Open

#142: add Automatic-Module-Name during build#469
SingingBush wants to merge 2 commits into
aws:mainfrom
SingingBush:feature/JPMS_automatic_module_name

Conversation

Copy link
Copy Markdown

Issue #, if available: #142

Description of changes: Support JPMS by using maven-jar-plugin to add Automatic-Module-Name during build

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

msailes commented Sep 3, 2024

Copy link
Copy Markdown
Contributor

Hi @SingingBush,

Would you be able to resolve the conflicts with this PR?

Thanks,

SingingBush force-pushed the feature/JPMS_automatic_module_name branch from 83e98d1 to 2ded191 Compare September 3, 2024 13:41

Copy link
Copy Markdown
Author

I'm looking into the failing tests and will push another commit

msailes requested review from maxday and smirnoal September 3, 2024 20:23
SingingBush force-pushed the feature/JPMS_automatic_module_name branch from 61dfd99 to c3c1cbb Compare October 3, 2024 22:52

Copy link
Copy Markdown
Author

I've rebased on the latest changes in main. Please re-run workflows. @maxday any chance of getting a second review

SingingBush force-pushed the feature/JPMS_automatic_module_name branch from c3c1cbb to 98ee838 Compare August 11, 2025 08:24

Copy link
Copy Markdown
Author

I've rebased again, please can someone run the workflow and review the changes

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL