Skip to content

Use ts-node for example - #344

Merged
kaisalmen merged 2 commits into
masterfrom
use-tsnode
Apr 8, 2022
Merged

Use ts-node for example#344
kaisalmen merged 2 commits into
masterfrom
use-tsnode

Conversation

@CGNonofr

@CGNonofr CGNonofr commented Apr 8, 2022

Copy link
Copy Markdown
Collaborator

No description provided.

@CGNonofr

CGNonofr commented Apr 8, 2022

Copy link
Copy Markdown
Collaborator Author

It looks like the build script is used by the ci, but isn't it useless everywhere else?

@CGNonofr

CGNonofr commented Apr 8, 2022

Copy link
Copy Markdown
Collaborator Author

Feel free to edit this PR @kaisalmen

@kaisalmen

Copy link
Copy Markdown
Collaborator

The build step is still meaningful to see if there are any code errors (and here the CI is useful, because it detects mistakes). vite/esbuild does not compile ts files, ts-node breaks, I guess

@CGNonofr

CGNonofr commented Apr 8, 2022

Copy link
Copy Markdown
Collaborator Author

vite/esbuild does not compile ts files, ts-node breaks, I guess

I'm not sure what you mean

I removed the 2 last commits

btw: there is a noEmit option for typescript to just check without emitting javascript files

@kaisalmen

Copy link
Copy Markdown
Collaborator

there is a noEmit option

Yeah, good idea.

I'm not sure what you mean

vite/esbuild does not invoke the tsc compiler. They just remove type information and produces javascript on the fly. That's why it is so fast, but in general before going to production you should everything compiles (no type issues, etc.)

@CGNonofr

CGNonofr commented Apr 8, 2022

Copy link
Copy Markdown
Collaborator Author

vite/esbuild does not invoke the tsc compiler. They just remove type information and produces javascript on the fly. That's why it is so fast, but in general before going to production you should everything compiles (no type issues, etc.)

got it!

@kaisalmen
kaisalmen merged commit 5d08fd0 into master Apr 8, 2022
@kaisalmen
kaisalmen deleted the use-tsnode branch July 10, 2026 10:40
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