Patricio Zavolinsky
Patricio Zavolinsky
Hi @avserban thanks for pointing this out, there was a missing `react-dom` dependency that is now required since react `16.x.x`. I've just pushed a new version of `react-cucumber` (`1.0.6`) pinning...
Hey @mrseanryan I removed the `NPM_TOKEN` secret while we did our investigation. I just re-enabled it, try again.
:+1: weird, that it didn't work before.
Hi @danvk, thanks for your comment! In your example, wouldn't running: ```sh ts-unused-exports tsconfig.json ``` have the desired results? (i.e. only report `unused` as unused) Cheers!
> what are you imagining is in `tsconfig.json`? My bad, I though you had a full `tsconfig.json` like this one: ```json { "files": [ "src/index.ts", "src/main.ts", "test/main_test.ts" ] } ```...
@wongjiahau I just merged one PR from @mrseanryan that flew under my radar changing the release process to GH Actions, sorry to use you as guinea pig, but could you...
Cool. I'll avoid a library if possible. I'd do something like: ```ts interface Options { // our current options specifying optional keys like this: thisOneIsOptional: string | undefined; // rather...
> what is wrong with using the `?` thing - I thought that would be equivalent? There is nothing wrong, and yes, both are almost equivalent, except the following: ```ts...
We should definitely explore the linter possibility. Specially now that we have TS support in ESLint. In raw TS, with `createProgram` we _can_ access the TypeChecker. I'm not sure if...