Skip to content

Conversation

cryptodev-2s
Copy link
Contributor

@cryptodev-2s cryptodev-2s commented Oct 17, 2025

Explanation

What is the current state of things and why does it need to change?

The publish-preview.yml GitHub Actions workflow is still using ts-node to execute the generate-preview-build-message.ts script. However, the project has recently migrated from ts-node to tsx PR

References

  • N/A

Checklist

  • I've updated the test suite for new or updated code as appropriate
    • No test changes needed for this workflow update
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
    • No documentation changes needed
  • I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
    • No changelog needed as this is an internal workflow change
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes
    • No breaking changes - internal workflow update only

Note

Switches the preview build message step in .github/workflows/publish-preview.yml from yarn ts-node to yarn tsx.

Written by Cursor Bugbot for commit 3f0fd29. This will update automatically on new commits. Configure here.

@cryptodev-2s cryptodev-2s self-assigned this Oct 17, 2025
@cryptodev-2s cryptodev-2s requested a review from a team as a code owner October 17, 2025 11:25
@cryptodev-2s cryptodev-2s enabled auto-merge (squash) October 17, 2025 11:30
@cryptodev-2s cryptodev-2s changed the title chore: replace ts-node with tsx in publish-preview workflow chore: replace ts-node with tsx in publish-preview workflow Oct 17, 2025
Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

LGTM!

Update the publish-preview GitHub Action workflow to use tsx instead of ts-node for running TypeScript scripts. This aligns with the project's migration from ts-node to tsx for better performance and ESM support.
@cryptodev-2s cryptodev-2s force-pushed the fix/replace-ts-node-with-tsx-in-workflow branch from 8aea010 to 3f0fd29 Compare October 17, 2025 12:55
@cryptodev-2s cryptodev-2s merged commit fc1c1c4 into main Oct 17, 2025
251 checks passed
@cryptodev-2s cryptodev-2s deleted the fix/replace-ts-node-with-tsx-in-workflow branch October 17, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants