Skip to content

chore: GitHub Action release and publish#255

Merged
lanesawyer merged 3 commits intomainfrom
lane/DT-9260-fix-github-action-publish
May 7, 2026
Merged

chore: GitHub Action release and publish#255
lanesawyer merged 3 commits intomainfrom
lane/DT-9260-fix-github-action-publish

Conversation

@lanesawyer
Copy link
Copy Markdown
Collaborator

What

Updates to GitHub Actions to get publishing to npm working

How

Makes a new release.yml that triggers the release of a new package version
Updates publish.yml to listen for the new tag made by the release action and publish to NPM

PR Checklist

  • Is your PR title following our conventional commit naming recommendations?
  • Have you filled in the PR Description Template?
  • Is your branch up to date with the latest in main?
  • Do the CI checks pass successfully?
  • Have you smoke tested the example applications?
  • Did you check that the changes meet accessibility standards?
  • Have you tested the application on these browsers?
    • Chrome (Fully supported)
    • Firefox (Major bug fixes supported)
    • Safari (Major bug fixes supported)

@lanesawyer lanesawyer requested a review from a team as a code owner May 6, 2026 20:15
@lanesawyer lanesawyer requested review from TheMooseman and chrisj May 6, 2026 20:15

- name: Publish
working-directory: ${{ steps.pkg.outputs.pkg_dir }}
run: npm publish --access public --provenance
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is an important change, we have to use npm for publishing because pnpm doesn't support the OIDC flow

@lanesawyer
Copy link
Copy Markdown
Collaborator Author

Going to give this a go!

@lanesawyer lanesawyer merged commit 6b715d1 into main May 7, 2026
5 checks passed
@lanesawyer lanesawyer deleted the lane/DT-9260-fix-github-action-publish branch May 7, 2026 18:37
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