Jacob Bandes-Storch

Results 278 comments of Jacob Bandes-Storch

Actually, in a larger repo I still don't get some type errors when changing files, even when I've changed all `import type` to `import` and enabled `"importsNotUsedAsValues": "preserve"`. I see...

Thanks for the response! Please let me know if you need any more information or help debugging 🙂 I noticed that my example repo demonstrates a partial issue, but the...

Thanks! Is there a new build of fork-ts-checker that I can install to test out #615 in my repo?

I tried 6.2.9, and improved some of the settings you recommended: https://github.com/foxglove/studio/compare/90cdcd3 However, I am now getting some errors when building. Strangely, the `Referenced project '...' may not disable emit`...

Here's an update to my example repo: https://github.com/jtbandes/ts-workspaces-repro/commit/a4dde59146fb29cb41c3136111bdec304907ceef I enabled `projectReferences: true` for ts-loader, and upgraded to fork-ts-checker 6.2.9. Now I still do not see new type errors when I...

I'm still having issues in my main repo that I unfortunately can't reproduce in the small test repo. If you have some time to try it out, we would really...

^ [This PR](https://github.com/foxglove/studio/pull/967) contains a workaround for the issue, so we now get type errors when changing type-only files and files in other workspace packages. However, it also seems to...

Does this mean ts-loader and fork-ts-checker are both competing to write .js and .d.ts files if project references are used? I noticed that I did get both `[tsl]` and regular...

I tried it briefly and updated this branch: https://github.com/foxglove/studio/compare/jacob/webpack-ts-fixes I started running into some new build errors but didn't have time to look into it yet (possibly I just missed...

> Or maybe you can just update `tsconfig.json` instead of using `configOverwrite` :D That's what I tried here: https://github.com/foxglove/studio/pull/967/files The problem is that these changes are only required because of...