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

Setup deployment to GitHub package registry for Gradle artifacts #26

Merged
merged 1 commit into from
Oct 25, 2020
Merged

Setup deployment to GitHub package registry for Gradle artifacts #26

merged 1 commit into from
Oct 25, 2020

Conversation

j4ckofalltrades
Copy link
Contributor

@j4ckofalltrades j4ckofalltrades commented Oct 25, 2020

Refs strongbox/strongbox#1927

Published artifact: https://github.com/j4ckofalltrades/strongbox-examples/packages/472460?version=1.0-SNAPSHOT

Note: Not sure if adding a description to artifact is possible at this time, I read through the maven-publish and GitHub docs but couldn't find anything.

@j4ckofalltrades
Copy link
Contributor Author

@carlspring Requesting review

Comment on lines 23 to 26
username = project.findProperty("gpr.user") ?: System.getenv("GITHUB_USERNAME")
password = project.findProperty("gpr.key") ?: System.getenv("GITHUB_TOKEN")
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to have Github secrets defined for these?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, GITHUB_USERNAME is sourced from actions/setup-java while GITHUB_TOKEN is automatically set by GitHub for each repository.

I'll document this here.

Copy link
Member

Choose a reason for hiding this comment

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

Excellent, thank you very much! :)

It might be worth mentioning all this in the master task for future reference.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that's a good idea. I'll drop in a comment there.

Copy link
Member

@carlspring carlspring left a comment

Choose a reason for hiding this comment

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

LGTM! Just a minor change in the URL for when we merge this in.

Thanks! :)

@carlspring carlspring merged commit 6f3bab0 into strongbox:master Oct 25, 2020
@carlspring
Copy link
Member

Thanks for your help! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants