As part of #2168, it was identified that only non-type checking TypeScript lint rules were running on wafer map code. The PR enabled TypeScript type-checking eslint rules but to avoid conflicts with existing PRs were not updated.
Update the paths in the file to be based on __dirname instead of how they behave now based on relative paths from the build output directory (like our other node JS scripts). Ssince the script will no longer be built from TS to JS it will become easier to reason about the scripts behavior as part of repo infra.
Remove the index.ts build script from tsconfig.json include and generate-workers/.eslintrc.js
This should change the generate-workers build output from having dist/esm/source/[files] to dist/esm/[files] so make sure that is accounted for (which it is desirable to not have a source directory in the build output, so that's good).
🧹 Tech Debt
As part of #2168, it was identified that only non-type checking TypeScript lint rules were running on wafer map code. The PR enabled TypeScript type-checking eslint rules but to avoid conflicts with existing PRs were not updated.
Acceptance criteria:
__dirname
instead of how they behave now based on relative paths from the build output directory (like our other node JS scripts). Ssince the script will no longer be built from TS to JS it will become easier to reason about the scripts behavior as part of repo infra.index.ts
build script from tsconfig.json include andgenerate-workers/.eslintrc.js
dist/esm/source/[files]
todist/esm/[files]
so make sure that is accounted for (which it is desirable to not have asource
directory in the build output, so that's good).