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

Chore: prepare publishing Maven artifacts to OSSRH and MavenCentral #1354

Merged
merged 17 commits into from May 25, 2022

Conversation

paullatzelsperger
Copy link
Member

@paullatzelsperger paullatzelsperger commented May 24, 2022

What this PR changes/adds

  • Switches over to using the GPG Agent for artifact signing
  • Injects the required POM information into all publications
  • configures the OSSRH repo URLs (snapshot and staging)

Why it does that

So that we can have lovely maven artifacts.

Further notes

As soon as this PR is merged we must update the JIPP (Eclipse Jenkins) build config to use main instead of this feature branch

Checklist

  • added appropriate tests?
  • performed checkstyle check locally?
  • added/updated copyright headers?
  • documented public classes/methods?
  • added/updated relevant documentation?
  • added relevant details to the changelog? (skip with label no-changelog)
  • formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 13:14 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 13:14 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 13:57 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 13:57 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 13:57 Inactive
@paullatzelsperger paullatzelsperger marked this pull request as draft May 24, 2022 13:57
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 14:01 Inactive
@codecov-commenter
Copy link

codecov-commenter commented May 24, 2022

Codecov Report

Merging #1354 (a5d7957) into main (db34d77) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1354      +/-   ##
============================================
- Coverage     67.68%   67.40%   -0.28%     
- Complexity        0     2885    +2885     
============================================
  Files           719      719              
  Lines         15859    15975     +116     
  Branches       1041     1052      +11     
============================================
+ Hits          10734    10768      +34     
- Misses         4650     4732      +82     
  Partials        475      475              
Impacted Files Coverage Δ
.../contract/offer/ContractDefinitionServiceImpl.java 100.00% <0.00%> (ø)
...taspaceconnector/azure/cosmos/CosmosDbApiImpl.java 0.00% <0.00%> (ø)
...store/cosmos/TransferProcessStoreCosmosConfig.java 0.00% <0.00%> (ø)
...re/cosmos/CosmosTransferProcessStoreExtension.java 0.00% <0.00%> (ø)
...on/store/CosmosContractNegotiationStoreConfig.java 0.00% <0.00%> (ø)
...store/CosmosContractNegotiationStoreExtension.java 0.00% <0.00%> (ø)
...dataspaceconnector/core/CoreServicesExtension.java 92.20% <0.00%> (+0.71%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db34d77...a5d7957. Read the comment docs.

@paullatzelsperger paullatzelsperger marked this pull request as ready for review May 24, 2022 14:12
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 14:12 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 14:13 Inactive
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
build.gradle.kts Outdated Show resolved Hide resolved
paullatzelsperger and others added 3 commits May 24, 2022 16:26
Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>
Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>
Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 24, 2022 14:27 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 06:26 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 06:26 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 06:35 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 06:35 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:03 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:03 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:04 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:09 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:30 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:30 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:30 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:52 Inactive
@paullatzelsperger paullatzelsperger temporarily deployed to Azure-dev May 25, 2022 07:53 Inactive
@paullatzelsperger
Copy link
Member Author

will merge regardless of failing checks. There is already a fix for them: #1343

@eclipse-edc-bot
Copy link
Contributor

Can one of the admins verify this patch?

@paullatzelsperger paullatzelsperger merged commit 9f06946 into main May 25, 2022
JurajKubelka added a commit to JurajKubelka/DataSpaceConnector that referenced this pull request May 26, 2022
* 'main' of github.com:JurajKubelka/DataSpaceConnector:
  refactor: remove basic launcher (eclipse-edc#1360)
  Chore: prepare publishing Maven artifacts to OSSRH and MavenCentral (eclipse-edc#1354)
  fix: load IdsTokenValidationServiceExtension properly (eclipse-edc#1352)
juliapampus added a commit to FraunhoferISST/edc-connector that referenced this pull request May 30, 2022
…clipse-edc#1354)

* add use gpg command

* Update build.gradle.kts

* Update build.gradle.kts

* select repo url dependin on SNAPSHOT

* changed OSSRH url

* attempt to include java code

* set pom to every publication

* added documentation

* added module list

* Update README.md

Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>

* Update README.md

Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>

* Update README.md

Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>

* updated modules file name

* trigger CI

* add clean-recreate on the modules list

* use absolute path for modules.md

* trigger CI

Co-authored-by: Julia Pampus <72392527+juliapampus@users.noreply.github.com>
@juliapampus juliapampus added this to In progress in Connector via automation May 31, 2022
@juliapampus juliapampus moved this from In progress to Done in Connector May 31, 2022
@paullatzelsperger paullatzelsperger deleted the chore/switch_to_gnupg_agent_for_signing branch July 6, 2022 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Connector
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants