react-scripts v3 via npm requires a peer of typescript@*

[kjg]

The inclusion of @typescript-eslint/eslint-plugin and @typescript-eslint/parser in react-scripts means there are now a lot of warnings about typescript being an unmet peer dependency when part of a project not using typescript.

I understand it is just a warning and I know I can ignore it, but I still wanted to bring it up. So many warnings coming from this package makes it harder to pull out the warnings I do want to fix.

[iansu]

That's a good point. I'm not sure if there's anything we can do about it unfortunately but we'll look into it.

[ianschmitz]

We ran into the same issue way back when adding TypeScript support when we added fork-ts-checker-webpack-plugin as a dependency. We solved it by working with their team to remove the peer dependency on typescript.

/cc @bradzacher do you think it would be reasonable to remove the peer dep on typescript in your packages?

[bradzacher]

I feel a good solution is that everyone should just use typescript 😄

But seriously, I don't think it's a huge deal if we remove them, considering we have it set to *.
Unless someone else on the core team has a problem, I'm happy to remove it.

cc @JamesHenry, @j-f1, @armano2

[JamesHenry]

Fine with me, we’ll make the update on our side

[ianschmitz]

typescript-eslint/typescript-eslint#443