Patrick McElhaney
Patrick McElhaney
@jarrod Are you sure changing `useDefineForClassFields` made a difference in your case? Currently `@web/dev-server-esbuild` doesn’t read `tsconfig.json` so it doesn’t make sense that changing that file would have any impact.
@jrencz I agree. I figured adding a config option would be easier to "sell" to the maintainers and is probably necessary in some cases anyway. All that remains now is...
Right after I hit the submit button it "finished". Node v16.14.2 on Mac OS 13.3.1. ``` ❯ npm test > [email protected] pretest > cd dev && npm i up to...
Tentatively scheduled for TypeScript 5.2. https://github.com/microsoft/TypeScript/issues/53461#issuecomment-1481760507
Here's a work-around until it's fixed: Install [patch-package](https://www.npmjs.com/package/patch-package), add `"postinstall": "patch-package"` to your npm scripts, and copy this code to a file named `./patches/tsutils+3.21.0.patch`. ```diff diff --git a/node_modules/tsutils/util/usage.js b/node_modules/tsutils/util/usage.js index...
I'm guessing it would need to be added to this file. https://github.com/storybookjs/test-runner/blob/next/src/util/getParsedCliOptions.ts
@btmills Thanks! Did you publish meeting minutes? Looking at the API, there may be a way to have our cake and eat it too with a small addition and no...
That’s exactly what I’m thinking. The `loc` defaults to the `node`’s location. Other information about the `node` is lost. I want to surface the `loc` of the `node.identifier` (when it...
Sorry, I missed your comment, @mdjermanovic. Removed that line so lint is no longer included in test. @nzakas This is ready to go as far as I know. Once it's...
That PR has been merged but there hasn't been a release since then. Pretty pretty please can you cut a new release? 😄