Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WP5 - Update fork-ts-checker-webpack-plugin [DRAFT] #10004

Closed
wants to merge 2 commits into from
Closed

WP5 - Update fork-ts-checker-webpack-plugin [DRAFT] #10004

wants to merge 2 commits into from

Conversation

raix
Copy link
Contributor

@raix raix commented Nov 3, 2020

Update [email protected]

References:

Part of making way for Webpack 5 #9994

yarn docker:e2e --test-suite simple not yet tested but done manual QA locally*

cc. @piotr-oles regarding your comment maybe you can help review this one?

Potential breaking changes if anyone externally depends on react-dev-utils:

  • react-dev-utils/typescriptFormatter - it's removed as it's no longer needed in the CRA project
  • react-dev-utils/WebpackDevServerUtils - createCompiler no longer use arguments devSocket or tscCompileOnError

Removed to keep the code base small and clean

@raix raix mentioned this pull request Nov 3, 2020
25 tasks
@raix raix changed the title WP5-Update fork-ts-checker-webpack-plugin WP5 - Update fork-ts-checker-webpack-plugin Nov 4, 2020
Copy link

@piotr-oles piotr-oles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, the only concern is issues hook tap :)

packages/react-dev-utils/WebpackDevServerUtils.js Outdated Show resolved Hide resolved
@jasonwilliams
Copy link
Contributor

https://github.com/TypeStrong/fork-ts-checker-webpack-plugin v6.0.1 is now out, might be worth bumping to that

@jasonwilliams
Copy link
Contributor

jasonwilliams commented Nov 14, 2020

@piotr-oles Thanks for the input - I'll try to resolve this in the weekend.
btw. I tried updating to v6.x but it started scanning node_modules (also added exclude patterns in both tsconfig.json and the overwrite but no change - not sure if there's an issue or just me :) )

We should raise that with them if it's doing that. It shouldn't be

@ianschmitz ianschmitz added this to the 4.1 milestone Dec 8, 2020
mbarrien added a commit to chanzuckerberg/czgenepi that referenced this pull request Feb 10, 2021
facebook/create-react-app#10004 makes changes to react-dev-utils
that are not present in the currently released version pulled down by npm. Revert and
reapply once new version of react-dev-utils is released (probably at same time as
create-react-app releases Webpack 5 compatible version).
@raix raix changed the title WP5 - Update fork-ts-checker-webpack-plugin WP5 - Update fork-ts-checker-webpack-plugin [DRAFT] Mar 20, 2021
@raix
Copy link
Contributor Author

raix commented Jul 1, 2021

Cherry picked into the wp5 branch - closing this pr

@raix raix closed this Jul 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants