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

Publish Maven artifacts to dogtagpki/repo #1013

Merged
merged 1 commit into from
Jul 9, 2024
Merged

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Jul 8, 2024

Previously JSS's Maven artifacts were published to GitHub Packages which is a private repository so it's difficult to use.

To resolve the problem, the pom.xml has been modified to publish the artifacts to a publicly accessible dogtagpki/repo instead.

https://github.com/dogtagpki/repo/wiki
https://github.com/dogtagpki/repo/wiki/Development-Guide

Once this PR is merged, the artifacts will be pushed into the maven branch like this:

https://github.com/edewata/repo/tree/maven
https://github.com/edewata/repo/commits/maven

Then I'll create a PR for LDAP SDK and PKI to use this repository.

@edewata edewata requested a review from fmarco76 July 8, 2024 22:01
Previously JSS's Maven artifacts were published to GitHub Packages
which is a private repository so it's difficult to use.

To resolve the problem, the pom.xml has been modified to publish
the artifacts to a publicly accessible dogtagpki/repo instead.
Copy link

sonarqubecloud bot commented Jul 9, 2024

Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

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

LGTM

@edewata
Copy link
Contributor Author

edewata commented Jul 9, 2024

@fmarco76 Thanks!

@edewata edewata merged commit e3a059a into dogtagpki:master Jul 9, 2024
32 of 34 checks passed
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.

2 participants