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

Migrate from jenkins to GitHub actions #231

Merged

Conversation

joakimen
Copy link
Contributor

@joakimen joakimen commented Feb 7, 2024

No description provided.

// We only run semantic-release on the release branches,
// as we do not want credentials to be exposed to the job
// on other branches or in PRs.
if (env.BRANCH_NAME ==~ /^(master|\d+\.(\d+|x)(\.x)?)$/) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Didn't include support for releasing from non-trunk branches, such as referenced here.
We don't have any such branches today, nor any plans on supporting it afaik

@joakimen joakimen marked this pull request as draft February 7, 2024 14:12
@joakimen
Copy link
Contributor Author

joakimen commented Feb 7, 2024

Before adding these changes, we need to decide how to handle public NPM packages, marking as draft for now.

@joakimen
Copy link
Contributor Author

joakimen commented Feb 9, 2024

Before adding these changes, we need to decide how to handle public NPM packages, marking as draft for now.

npm token strategy has been decided, and token has been added to workflow according to the semrel guide

@joakimen joakimen marked this pull request as ready for review February 9, 2024 08:51
Copy link
Contributor

@stekern stekern left a comment

Choose a reason for hiding this comment

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

A couple of comments regarding job permissions, but otherwise looks good 🚀

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Show resolved Hide resolved
@joakimen joakimen requested a review from stekern February 9, 2024 10:00
Copy link
Contributor

@stekern stekern left a comment

Choose a reason for hiding this comment

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

LGTM!

I suggest we clean up the commit history a bit, and use a conventional commit type (e.g., fix(ci): use github actions or feat(ci): use github actions) that will trigger a release so we can verify that everything works as expected upon merge.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@joakimen joakimen force-pushed the jol/CALS-705/migrate-from-jenkins-to-github-actions branch from c909931 to c1fb632 Compare February 9, 2024 13:25
@joakimen joakimen merged commit 63118aa into master Feb 9, 2024
3 checks passed
@joakimen joakimen deleted the jol/CALS-705/migrate-from-jenkins-to-github-actions branch February 9, 2024 13:30
Copy link

github-actions bot commented Feb 9, 2024

🎉 This PR is included in version 2.19.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants