Skip to content
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

build: automate publishing to Maven Central #11731

Merged
merged 11 commits into from
May 13, 2024

Conversation

jjohannes
Copy link
Collaborator

@jjohannes jjohannes commented Feb 26, 2024

Description:

As a side effect, assemble and qualityGate now also build the Javadoc and find Javadoc errors.
This PR fixes the errors that were found.

Note: new/reworked convention plugin files follow the naming pattern that will be used consistently with #12997

Related issue(s):

Fixes #11237

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@jjohannes jjohannes requested review from a team as code owners February 26, 2024 10:57
@jjohannes jjohannes changed the title 11237 maven central publish build: automate publishing to Maven Centra Feb 26, 2024
@jjohannes jjohannes changed the title build: automate publishing to Maven Centra build: automate publishing to Maven Central Feb 26, 2024
@jjohannes jjohannes self-assigned this Feb 26, 2024
Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Restart) Results

2 tests   2 ✅  5m 23s ⏱️
2 suites  0 💤
2 files    0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Node Death Reconnect) Results

2 tests   2 ✅  6m 55s ⏱️
2 suites  0 💤
2 files    0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Token) Results

235 tests   233 ✅  19m 20s ⏱️
 17 suites    2 💤
 17 files      0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Crypto) Results

335 tests   335 ✅  42m 45s ⏱️
 25 suites    0 💤
 25 files      0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: E2E Test Results

    1 files      1 suites   25m 26s ⏱️
311 tests 311 ✔️ 0 💤 0
333 runs  333 ✔️ 0 💤 0

Results for commit c4310bf.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Misc) Results

459 tests   449 ✅  43m 49s ⏱️
 77 suites   10 💤
 77 files      0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Time Consuming) Results

21 tests   21 ✅  54m 30s ⏱️
 3 suites   0 💤
 3 files     0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: HAPI Test (Smart Contract) Results

592 tests   592 ✅  1h 19m 1s ⏱️
 63 suites    0 💤
 63 files      0 ❌

Results for commit b7c3977.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: Integration Test Results

    5 files      5 suites   1h 6m 34s ⏱️
322 tests 322 ✔️ 0 💤 0
333 runs  333 ✔️ 0 💤 0

Results for commit c4310bf.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 26, 2024

Node: Unit Test Results

  2 280 files  ±0    2 280 suites  ±0   3h 28m 22s ⏱️ + 1h 8m 50s
118 804 tests +2  118 737 ✅ +2  67 💤 ±0  0 ❌ ±0 
127 289 runs  +2  127 222 ✅ +2  67 💤 ±0  0 ❌ ±0 

Results for commit b7c3977. ± Comparison against base commit 936032a.

This pull request removes 3976 and adds 3741 tests. Note that renamed tests count towards both.

  
             IssuerDN: CN=s-aaaa
            SubjectDN: CN=s-aaaa
           Final Date: Fri Jan 01 00:00:00 UTC 2100
           Public Key: RSA Public Key [2e:28:bc:1e:d3:83:25:92:8e:cb:98:b1:b6:84:06:9c:d5:d8:14:d5],[56:66:d1:a4]
           Start Date: Sat Jan 01 00:00:00 UTC 2000
         SerialNumber: 12482092706667292405
        modulus: c1a0ff5d2372b53d12d12bb87dd03f5e…
        modulus: c1a0ff5d2372b53d12d12bb87dd03f5…
…
com.hedera.node.app.grpc.impl.netty.GrpcServiceBuilderTest ‑ [4] 

com.hedera.node.app.grpc.impl.netty.GrpcServiceBuilderTest ‑ [6] 

com.hedera.node.app.grpc.impl.netty.GrpcServiceBuilderTest ‑ [7]   
  
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [10] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@dae8ada2
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [11] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@8304875c
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [12] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@bdb88569
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [13] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@40fa07b9
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [14] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@e15d02d
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [15] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@5894eee2
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [16] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@8069ed5
…

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.80%. Comparing base (f2ac254) to head (fa07dce).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop   #11731      +/-   ##
=============================================
- Coverage      61.80%   61.80%   -0.01%     
+ Complexity     30620    30619       -1     
=============================================
  Files           3382     3382              
  Lines         138320   138320              
  Branches       14528    14528              
=============================================
- Hits           85488    85485       -3     
- Misses         49133    49136       +3     
  Partials        3699     3699              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

jjohannes and others added 10 commits May 10, 2024 10:53
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
...to make the cross-project 'task.dependsOn' setup working without
having to list all projects that are published (again).

Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
Signed-off-by: Nathan Klick <nathan@swirldslabs.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>
nathanklick
nathanklick previously approved these changes May 10, 2024
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
@Nana-EC Nana-EC requested review from david-bakin-sl and removed request for quiet-node May 10, 2024 18:01
Copy link
Member

@Neeharika-Sompalli Neeharika-Sompalli left a comment

Choose a reason for hiding this comment

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

LGTM from hedera-base

Copy link
Member

@david-bakin-sl david-bakin-sl left a comment

Choose a reason for hiding this comment

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

LGTM, only comment I'd like to seen an answer to is why cli-clients isn't published. I don't know the issues - but those are tools useful to developers, so not knowing any reason to say no, I say instead: why not publish them? Let me know.

@kimbor kimbor mentioned this pull request May 10, 2024
@jjohannes jjohannes merged commit 9f5c168 into develop May 13, 2024
44 of 46 checks passed
@jjohannes jjohannes deleted the 11237-maven-central-publish branch May 13, 2024 05:58
stoyanov-st pushed a commit that referenced this pull request May 15, 2024
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
Co-authored-by: Nathan Klick <nathan@swirldslabs.com>
Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>
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.

Add support for automatic Maven Central releases
9 participants