Skip to content

Deploy to npm tag from branch #221

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

Closed
wants to merge 9 commits into from
Closed

Deploy to npm tag from branch #221

wants to merge 9 commits into from

Conversation

cwillisf
Copy link
Contributor

Proposed Changes

Main change: allow the "Publish" workflow to publish to a tag other than latest, depending on the branch name and the state of the "Prerelease" checkbox in GitHub's release UI.

Additional changes:

  • commitlint now runs as a separate workflow instead of as part of the CI workflow, so you can tell the difference between a commitlint failure vs. a build/test failure.
  • the release commit message now follows Conventional Commit style.

Reason for Changes

This allows the spork branch to publish to the spork tag on npm, making it easier to test on staging. Other branches are likely to want this soon, so I pulled this out of the spork branch so we don't need to wait for all the spork QA to finish before those other branches can use this change.

Test Coverage

I've already used this workflow from the spork branch.

@cwillisf cwillisf marked this pull request as ready for review March 24, 2025 15:28
@cwillisf cwillisf requested a review from a team as a code owner March 24, 2025 15:28
@cwillisf
Copy link
Contributor Author

Closing in favor of #222

@cwillisf cwillisf closed this Mar 24, 2025
@cwillisf cwillisf deleted the deploy-from-branch branch March 24, 2025 20:52
@github-actions github-actions bot locked and limited conversation to collaborators Mar 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant