Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

Automated deployment #564

Merged
merged 7 commits into from
Oct 26, 2020
Merged

Automated deployment #564

merged 7 commits into from
Oct 26, 2020

Conversation

jponge
Copy link
Contributor

@jponge jponge commented Oct 24, 2020

Automated CI/CD pipeline to Sonatype OSS

A signing key and gradle.properties with credentials are encrypted.

@jponge jponge added this to the 3.4.0 milestone Oct 24, 2020
@jponge jponge requested review from yloiseau and k33g October 24, 2020 15:40
@jponge
Copy link
Contributor Author

jponge commented Oct 24, 2020

⚠️ Please do not merge until we have been given access to the repository secrets settings by the Eclipse Foundation.

@jponge
Copy link
Contributor Author

jponge commented Oct 24, 2020

@k33g @yloiseau Please have a look, especially regarding security

Copy link
Contributor

@k33g k33g left a comment

Choose a reason for hiding this comment

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

LGTM (with my understanding of actions) 😉

.build/deploy.sh Outdated

echo "🧹 Cleanup..."

rm gradle.properties golo-dev-sign.asc
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add a trap to clean even if the script is aborted ?
Since you have set -e, if the ./gradlew publish fails, the script stops without cleaning...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

How would you do that?

Copy link
Contributor

Choose a reason for hiding this comment

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

$ help trap 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're the expert, fancy a commit on this branch? 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@jponge jponge requested a review from yloiseau October 26, 2020 10:15
@jponge jponge merged commit e3e4934 into master Oct 26, 2020
@jponge jponge deleted the infra/ci-rework branch October 26, 2020 10:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants