Run all TS code (including examples/jsm) through tsc when linting.

[MichaelBuerge]

#17409

[MichaelBuerge]

The added dev dependency on @types/webgl2 is to let tsc know about WebGL2RenderingContext appearing in
https://github.com/mrdoob/three.js/blob/dev/src/renderers/webgl/WebGLAnimation.d.ts
https://github.com/mrdoob/three.js/blob/dev/src/renderers/webgl/WebGLAttributes.d.ts
https://github.com/mrdoob/three.js/blob/dev/src/renderers/webgl/WebGLUtils.d.ts

Not sure what the status is of these.
They were not passed to tsc before, as the previous entry point was src/Three.d.ts (which doesn't include the above files), whereas now all TS files are run through tsc.

[gkjohnson]

Not sure what the status is of these.
They were not passed to tsc before, as the previous entry point was src/Three.d.ts (which doesn't include the above files), whereas now all TS files are run through tsc.

Those files are included by renderers/WebGLRenderer.d.ts, though? Does tsc not recursively validate the files?

[MichaelBuerge]

Not sure what the status is of these.
They were not passed to tsc before, as the previous entry point was src/Three.d.ts (which doesn't include the above files), whereas now all TS files are run through tsc.

Those files are included by renderers/WebGLRenderer.d.ts, though?

No they're not. WebGLRenderer.js yes, but not WebGLRenderer.d.ts.
Furthermore, the include of WebGLUtils.d.ts in src/Three.d.ts is commented out :

three.js/src/Three.d.ts

Line 175 in f3205d7

//export * from './renderers/webgl/WebGLUtils';

Does tsc not recursively validate the files?

Yes it does. Though only the ones it gets to from the entry points. The entry point was src/Three.d.ts, which due to the commented import left out the files I mentioned.

[Mugen87]

Looks good! I think we should give this a try 👍

[mrdoob]

Thanks!

[MichaelBuerge]

Thanks!

You're welcome!