@rollup/plugin-typescript noEmitOnError should default to true

[gempain]

• Rollup Plugin Name: @rollup/plugin-typescript
• Rollup Plugin Version: 11.0.0 and before

Expected Behavior / Situation

noEmitOnError should be true by default. It makes no sense to emit code when the Typescript build fails. It can lead to production code not working as expected.

Actual Behavior / Situation

By default the value is false.

Modification Proposal

Change the value from false to true.

[pimartin]

It looks like it was introduced for the watch mode.

Maybe a slightly more complex default could be used: false when in watch mode, true otherwise ?

[stale]

Hey folks. This issue hasn't received any traction for 60 days, so we're going to close this for housekeeping. If this is still an ongoing issue, please do consider contributing a Pull Request to resolve it. Further discussion is always welcome even with the issue closed. If anything actionable is posted in the comments, we'll consider reopening it. ⓘ

[simlmx]

I agree with the previous comments: the default should be to error when building. But when watching it cannot crash the watch otherwise what's the point.

For me the plugin is unusable if it can not both error on build and not crash on watch.