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

test(integrations): switch react-scripts to webpack #8329

Merged
merged 5 commits into from
Nov 24, 2024
Merged

Conversation

lachlancollins
Copy link
Member

@lachlancollins lachlancollins commented Nov 23, 2024

CRA (more specifically react-scripts) installs hundreds of old packages which often have conflicts with newer dependencies. This leads to otherwise simple dependency updates causing major headaches. One temporary solution was #7667, however newer updates are running into infuriating babel resolution issues. This PR cuts out react-scripts and tests directly against webpack.

Copy link

nx-cloud bot commented Nov 23, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit acc09dc. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

Copy link

pkg-pr-new bot commented Nov 23, 2024

Open in Stackblitz

More templates

@tanstack/angular-query-devtools-experimental

pnpm add https://pkg.pr.new/@tanstack/angular-query-devtools-experimental@8329

@tanstack/query-async-storage-persister

pnpm add https://pkg.pr.new/@tanstack/query-async-storage-persister@8329

@tanstack/query-broadcast-client-experimental

pnpm add https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@8329

@tanstack/angular-query-experimental

pnpm add https://pkg.pr.new/@tanstack/angular-query-experimental@8329

@tanstack/query-core

pnpm add https://pkg.pr.new/@tanstack/query-core@8329

@tanstack/eslint-plugin-query

pnpm add https://pkg.pr.new/@tanstack/eslint-plugin-query@8329

@tanstack/query-devtools

pnpm add https://pkg.pr.new/@tanstack/query-devtools@8329

@tanstack/query-persist-client-core

pnpm add https://pkg.pr.new/@tanstack/query-persist-client-core@8329

@tanstack/query-sync-storage-persister

pnpm add https://pkg.pr.new/@tanstack/query-sync-storage-persister@8329

@tanstack/react-query

pnpm add https://pkg.pr.new/@tanstack/react-query@8329

@tanstack/react-query-devtools

pnpm add https://pkg.pr.new/@tanstack/react-query-devtools@8329

@tanstack/react-query-next-experimental

pnpm add https://pkg.pr.new/@tanstack/react-query-next-experimental@8329

@tanstack/react-query-persist-client

pnpm add https://pkg.pr.new/@tanstack/react-query-persist-client@8329

@tanstack/solid-query

pnpm add https://pkg.pr.new/@tanstack/solid-query@8329

@tanstack/solid-query-devtools

pnpm add https://pkg.pr.new/@tanstack/solid-query-devtools@8329

@tanstack/solid-query-persist-client

pnpm add https://pkg.pr.new/@tanstack/solid-query-persist-client@8329

@tanstack/svelte-query

pnpm add https://pkg.pr.new/@tanstack/svelte-query@8329

@tanstack/svelte-query-devtools

pnpm add https://pkg.pr.new/@tanstack/svelte-query-devtools@8329

@tanstack/svelte-query-persist-client

pnpm add https://pkg.pr.new/@tanstack/svelte-query-persist-client@8329

@tanstack/vue-query

pnpm add https://pkg.pr.new/@tanstack/vue-query@8329

@tanstack/vue-query-devtools

pnpm add https://pkg.pr.new/@tanstack/vue-query-devtools@8329

commit: acc09dc

Copy link

codecov bot commented Nov 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.95%. Comparing base (2588a08) to head (acc09dc).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           main    #8329       +/-   ##
=========================================
+ Coverage      0   45.95%   +45.95%     
=========================================
  Files         0      200      +200     
  Lines         0     7507     +7507     
  Branches      0     1713     +1713     
=========================================
+ Hits          0     3450     +3450     
- Misses        0     3680     +3680     
- Partials      0      377      +377     
Components Coverage Δ
@tanstack/angular-query-devtools-experimental ∅ <ø> (∅)
@tanstack/angular-query-experimental 87.13% <ø> (∅)
@tanstack/eslint-plugin-query 88.12% <ø> (∅)
@tanstack/query-async-storage-persister 43.85% <ø> (∅)
@tanstack/query-broadcast-client-experimental ∅ <ø> (∅)
@tanstack/query-codemods 0.00% <ø> (∅)
@tanstack/query-core 93.69% <ø> (∅)
@tanstack/query-devtools 4.78% <ø> (∅)
@tanstack/query-persist-client-core 57.73% <ø> (∅)
@tanstack/query-sync-storage-persister 84.61% <ø> (∅)
@tanstack/react-query 95.54% <ø> (∅)
@tanstack/react-query-devtools 10.00% <ø> (∅)
@tanstack/react-query-next-experimental ∅ <ø> (∅)
@tanstack/react-query-persist-client 100.00% <ø> (∅)
@tanstack/solid-query 78.20% <ø> (∅)
@tanstack/solid-query-devtools ∅ <ø> (∅)
@tanstack/solid-query-persist-client 100.00% <ø> (∅)
@tanstack/svelte-query 87.33% <ø> (∅)
@tanstack/svelte-query-devtools ∅ <ø> (∅)
@tanstack/svelte-query-persist-client 100.00% <ø> (∅)
@tanstack/vue-query 71.45% <ø> (∅)
@tanstack/vue-query-devtools ∅ <ø> (∅)
---- 🚨 Try these New Features:

@lachlancollins lachlancollins changed the title chore(deps): update packages chore: switch CRA 4 to webpack 4 Nov 24, 2024
@lachlancollins lachlancollins changed the title chore: switch CRA 4 to webpack 4 test(integrations): switch CRA 4 to webpack 4 Nov 24, 2024
@lachlancollins lachlancollins marked this pull request as ready for review November 24, 2024 00:55
@lachlancollins lachlancollins changed the title test(integrations): switch CRA 4 to webpack 4 test(integrations): switch react-scripts to webpack Nov 24, 2024
@lachlancollins lachlancollins merged commit b846c53 into main Nov 24, 2024
8 checks passed
@lachlancollins lachlancollins deleted the update-deps branch November 24, 2024 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant