Turbo-ize

[lishaduck]

Depends on #163
Relates to #156

npm-run-all kept acting up, so I went ahead and migrated elm-review to turbo.
Key things: everything is parallelized now. It's also cached.

To run tasks, use turbo ... instead of npm run ..., which is installable via npm install -g turbo
You can also use npx turbo. They both run using our pinned turbo, the global one just proxies to the local one.
And, of course, you can still run npm run ..., it just won't get cached.

If you want to add a task, just add it in the package.json. It'll automatically get cached. If you find it gets invalidated too much, you can add it to turbo.json.

Here's the diff as GH doesn't play well with stacks from forks: lishaduck/node-elm-review@bump-deps-2...turbo
I also updated CI. I hope you don't mind.

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@types/[email protected]	None	0	7.09 kB	types
npm/@types/[email protected]	Transitive: environment, eval	+12	447 kB	types
npm/@types/[email protected]	None	0	6.27 kB	types
npm/@types/[email protected]	None	0	1.61 MB	types
npm/@typescript-eslint/[email protected]	Transitive: environment, filesystem	+42	6.99 MB	jameshenry
npm/@typescript-eslint/[email protected]	Transitive: environment, filesystem	+30	2.14 MB	jameshenry
npm/[email protected]	Transitive: environment, filesystem, shell	+8	489 kB	lydell
npm/[email protected]	environment, filesystem, network, shell	0	100 kB	lydell
npm/[email protected]	None	0	20.8 kB	lydell
npm/[email protected]	Transitive: environment, eval, filesystem, shell	+2	477 kB	turbobot
npm/[email protected]	Transitive: environment, eval, filesystem, unsafe	+71	10.9 MB	sindresorhus
npm/[email protected]	environment, filesystem Transitive: eval, unsafe	+75	11.1 MB	eslintbot

🚮 Removed packages: npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

[jfmengels]

Thank you for the PR. I'm willing to try turbo out.

Can you tell me where I can find the docs for turbo? I'm having trouble finding it.
It's at https://turbo.build/repo/docs
I kept getting confused because I though turbo was a different tool than turborepo.

Do you also know if there's a way to not have the interactive window showing up? Mostly out of curiosity.

npm-run-all kept acting up

Do you remember what kind of problems you were encountering?

[jfmengels]

Do you know if it's possible to keep a test script so that I can just run npm test ?

[lishaduck]

Can you tell me where I can find the docs for turbo? I'm having trouble finding it. It's at https://turbo.build/repo/docs I kept getting confused because I though turbo was a different tool than turborepo.

Yeah. turbo is the cli, which came first. Then they made turbopack, so they retroactively named it turborepo, but they share a cli (well, they will eventually).

Do you also know if there's a way to not have the interactive window showing up? Mostly out of curiosity.

The --log-order flag. You can pass either --log-order=grouped or --log-order=stream.
stream is essentially npm-run-all

npm-run-all kept acting up

Do you remember what kind of problems you were encountering?

It wasn't injecting node_modules/.bin into the path, so some tests were failing.

Do you know if it's possible to keep a test script so that I can just run npm test ?

Yeah. I can rename test to, say, testing, and then alias npm test to turbo testing. It's meant to be used in a monorepo where you do that for all your tasks in your root, then use turbo in the individual packages, but it works well in single package repos as well.

[jfmengels]

I can rename test to, say, testing, and then alias npm test to turbo testing

I think this would be great. npm test (or actually npm t) is a very ingrained shortcut for me 😄

I've pushed the change now 👍

[lishaduck]

If you prefer the log-order flags, I just found out you can set an env variable:
Set export TURBO_UI=0 and export TURBO_LOG_ORDER=streamed (or grouped) in your .zshrc (or .bash_profile or whatever)