fix: Correct the 'build' command in the package.json file.

[mishengqiang]

The package.json file is located at template/config/typescript/package.json.

run-p type-check \"build-only {@}\" -- -> run-p type-check \"build-only -- {@}\" --

#393

[cexbrayat]

It fixes commands like npm run build -- --sourcemap, but it no longer works with pnpm run build --sourcemap because it adds an extra --

@sodatea WDYT?

[mishengqiang]

Ah, I didn't consider pnpm and yarn. After testing, yarn performs the best – it passes parameters as expected regardless of whether -- is added or not. npm requires an extra -- to pass parameters as expected. pnpm doesn't need an extra -- to pass parameters as expected.

The image below demonstrates when an extra '--' is added.

[cexbrayat]

Yes, that's why I don't know if we should change the default or not, or maybe there is a way to make it work with all package managers?

[haoqunjiang]

Yeah, I think this should be resolved in the upstream npm-run-all2 package instead.
Before that, maybe we can add a note in the generated README?

[ayu-exorcist]

Yeah, I think this should be resolved in the upstream npm-run-all2 package instead. Before that, maybe we can add a note in the generated README?

I resolved it in the bcomnes/npm-run-all2#143.

We can run run-p type-check \"build-only -- {@}\" -- on npm/yarn/pnpm when the PR is merged

[ayu-exorcist]

Yeah, I think this should be resolved in the upstream npm-run-all2 package instead. Before that, maybe we can add a note in the generated README?

I resolved it in the bcomnes/npm-run-all2#143.

We can run run-p type-check \"build-only -- {@}\" -- on npm/yarn/pnpm when the PR is merged

Already merged in npm-run-all2 v6.2.1

We can try to merge this PR now!

[bcomnes]

Turns out this was a breaking change so I reverted. I need to consider this change a little bit more and at a minimum, put it out in a major release with notes on upgrading.