-
Notifications
You must be signed in to change notification settings - Fork 105
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
Full jigsaw adoption / distribution changes #1292
Merged
Merged
Conversation
This file contains 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
Contributor
melowe
commented
May 19, 2021
•
edited
Loading
edited
- Remove maven
- Create zip/tar dists not shaded jars
- Added module-infos to all modules
- Restrict access to concrete types using module-info
- Upgraded gradle ro 7.0.2
- Refactored key vault modules.
- Use mockito-inline allowing static mocking and mocking of final classes
- Kill off dead boiler plate code required to use ServiceLoader with java 8
- Remove spring dependency for core code (still used in key vault client libs)
- Change older javax.* group ids to jakarta.*
- Remove deprecated configuration (config/server and config/unixfile)
- Changed kalium encryptor module to serve as a custom encryptor sample rather than using the NACL
- Add hk2 binder that supports javax.inject for jersey runtime
… Move stream consumer to more central place and created exec managers with correct jvm args
This reverts commit a910315
…e it will interpret the wildcard
…ing run inside of the source tree..
…anup and stricter assertion.
AzureITs currently fail and need to be reworked to pass with the new client library. These tests are already excluded in the Actions workflows.
Test fails in Actions. PartyInfo sharing is covered by other tests, this test only needs to verify there is a ThirdParty API response.
…e running atests against matching quorum version
762171c
to
900fd5c
Compare
Krish1979
previously approved these changes
Jun 15, 2021
…and jnacl dists in itests as currently tests are not being run against the release distribution.
…le that becomes unused
Krish1979
approved these changes
Jun 15, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.