Skip to content

Migrate bin/*.js to TypeScript, use tsx #2866

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

Kocal
Copy link
Member

@Kocal Kocal commented Jun 25, 2025

Q A
Bug fix? no
New feature? no
Docs? no
Issues Fix #...
License MIT

Types ✨

Following #2865, in fact I didn't go with Node & --experimental-strip-types flag, but with https://tsx.is/ instead.

@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Jun 25, 2025
@@ -1,6 +1,7 @@
{
"private": true,
"packageManager": "yarn@4.5.0",
"type": "module",
Copy link
Member Author

Choose a reason for hiding this comment

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

It should not impact src/**/assets packages

@Kocal Kocal force-pushed the migrate-bin-files-from-js-to-ts branch from d2ac9ff to 640ebe5 Compare June 25, 2025 19:18
@Kocal Kocal force-pushed the migrate-bin-files-from-js-to-ts branch from 640ebe5 to de41146 Compare June 25, 2025 21:13
@smnandre
Copy link
Member

Trusting you here :)

@Kocal
Copy link
Member Author

Kocal commented Jun 26, 2025

I noticed using tsx for building UX packages slowed down the pipeline by ~17 seconds, so I'm not really sure about this PR... but maybe it can be "compensated" by replacing Rollup with Rolldown in another PR (I still have something to see with React and react-dom/client dependency thing)

I may try to give to another change to node --experimental-strip-types, or maybe use Node.js 24 for internal tools instead of 22.6. 🤷🏻

@Kocal Kocal marked this pull request as draft June 27, 2025 16:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants