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
Cleanup Module patching #8561
Cleanup Module patching #8561
Conversation
4a028f1
to
72773e7
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #8561 +/- ##
==========================================
Coverage ? 65.25%
Complexity ? 29680
==========================================
Files ? 3262
Lines ? 124594
Branches ? 12922
==========================================
Hits ? 81310
Misses ? 40208
Partials ? 3076 ☔ View full report in Codecov by Sentry. |
55ac913
to
d695cfa
Compare
Kudos, SonarCloud Quality Gate passed! |
d695cfa
to
5071fbb
Compare
SonarCloud Quality Gate failed. 0 Bugs No Coverage information Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
15bb5ae
to
7f9c5e5
Compare
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.
approving the two files that are co-owned by hashgraph team.
Once #8455 is merged, I will rebase this again and address the conflicts and problem. |
779358b
to
b6c52b5
Compare
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
b6c52b5
to
d7cc35f
Compare
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com> Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Description:
com.hedera.hashgraph.jpms-modules.gradle.kts
data/libs
to be real Modulesdata/lib
data/lib
has amodule-info.class
. This can be extended later to check more precisely what we expect/allow indata/lib
.The patched modules might not yet be ready to really run on the module path. That is, they might miss certain "requires" at runtime and there could be issue with Jars containing native code. Hence this PR does not change how we start/package the application. However, in a follow up this can be addressed and then also
jlink
andjpackage
could be used.Related issue(s):
Notes for reviewer:
Checklist