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

Add support for jsconfig.json resolution #3409

Merged
merged 2 commits into from
Jan 21, 2023
Merged

Add support for jsconfig.json resolution #3409

merged 2 commits into from
Jan 21, 2023

Conversation

jridgewell
Copy link
Contributor

@jridgewell jridgewell commented Jan 20, 2023

There doesn't appear to be any issues with just returning a jsconfig.json when we're looking for a tsconfig.json (provided one didn't exist). Finding this gives a TsConfigReferenceVc (used for finding type definitions only) and enables the paths/baseUrl resolution options.

Fixes WEB-230

There doesn't appear to be any issues with just returning a `jsconfig.json` when we're looking for a `tsconfig.json` (provided one didn't exist). Finding this a `TsConfigReferenceVc` (used for finding type definitions only), and enable the `paths`/`baseUrl` resolution options.
@jridgewell jridgewell requested a review from a team as a code owner January 20, 2023 23:17
@vercel
Copy link

vercel bot commented Jan 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
examples-basic-web 🔄 Building (Inspect) Jan 21, 2023 at 1:35AM (UTC)
examples-native-web 🔄 Building (Inspect) Jan 21, 2023 at 1:35AM (UTC)
8 Ignored Deployments
Name Status Preview Comments Updated
examples-cra-web ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)
examples-designsystem-docs ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)
examples-kitchensink-blog ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)
examples-nonmonorepo ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)
examples-svelte-web ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)
examples-tailwind-web ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)
turbo-site ⬜️ Ignored (Inspect) Visit Preview Jan 21, 2023 at 1:35AM (UTC)
turbo-vite-web ⬜️ Ignored (Inspect) Jan 21, 2023 at 1:35AM (UTC)

@github-actions
Copy link
Contributor

Benchmark for f0396cf

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 8215.03µs ± 20.87µs 8224.85µs ± 34.76µs +0.12%
bench_hmr_to_commit/Turbopack RCC/1000 modules 8543.08µs ± 77.22µs 8481.74µs ± 70.75µs -0.72%
bench_hmr_to_commit/Turbopack RSC/1000 modules 441.16ms ± 1.74ms 444.83ms ± 2.70ms +0.83%
bench_hmr_to_commit/Turbopack SSR/1000 modules 8351.02µs ± 67.90µs 8360.24µs ± 58.13µs +0.11%
bench_hmr_to_eval/Turbopack CSR/1000 modules 7335.35µs ± 55.19µs 7237.64µs ± 63.37µs -1.33%
bench_hmr_to_eval/Turbopack RCC/1000 modules 7418.05µs ± 51.69µs 7394.31µs ± 54.33µs -0.32%
bench_hmr_to_eval/Turbopack SSR/1000 modules 7355.51µs ± 61.13µs 7493.36µs ± 134.14µs +1.87%
bench_hydration/Turbopack RCC/1000 modules 3283.87ms ± 9.27ms 3315.59ms ± 8.31ms +0.97%
bench_hydration/Turbopack RSC/1000 modules 2755.54ms ± 7.28ms 2773.70ms ± 8.87ms +0.66%
bench_hydration/Turbopack SSR/1000 modules 2562.65ms ± 6.46ms 2578.10ms ± 8.63ms +0.60%
bench_startup/Turbopack CSR/1000 modules 1607.34ms ± 4.29ms 1603.58ms ± 9.37ms -0.23%
bench_startup/Turbopack RCC/1000 modules 2448.75ms ± 6.63ms 2465.06ms ± 6.50ms +0.67%
bench_startup/Turbopack RSC/1000 modules 2344.68ms ± 10.68ms 2368.55ms ± 5.78ms +1.02%
bench_startup/Turbopack SSR/1000 modules 2020.47ms ± 4.59ms 2021.44ms ± 1.99ms +0.05%

@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2023

🟢 CI successful 🟢

Thanks

@jridgewell jridgewell added the pr: automerge Kodiak will merge these automatically after checks pass label Jan 21, 2023
@github-actions
Copy link
Contributor

Benchmark for b6f668b

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 8679.35µs ± 59.30µs 8752.04µs ± 54.28µs +0.84%
bench_hmr_to_commit/Turbopack RCC/1000 modules 8827.48µs ± 79.91µs 8841.62µs ± 51.18µs +0.16%
bench_hmr_to_commit/Turbopack RSC/1000 modules 464.70ms ± 1.54ms 461.60ms ± 2.20ms -0.67%
bench_hmr_to_commit/Turbopack SSR/1000 modules 8660.58µs ± 57.38µs 8798.54µs ± 58.76µs +1.59%
bench_hmr_to_eval/Turbopack CSR/1000 modules 7710.52µs ± 71.17µs 7630.85µs ± 41.52µs -1.03%
bench_hmr_to_eval/Turbopack RCC/1000 modules 7777.99µs ± 92.01µs 7708.98µs ± 53.83µs -0.89%
bench_hmr_to_eval/Turbopack SSR/1000 modules 7697.87µs ± 76.64µs 7722.27µs ± 98.65µs +0.32%
bench_hydration/Turbopack RCC/1000 modules 3428.32ms ± 14.75ms 3415.20ms ± 7.96ms -0.38%
bench_hydration/Turbopack RSC/1000 modules 2847.66ms ± 7.80ms 2881.85ms ± 11.45ms +1.20%
bench_hydration/Turbopack SSR/1000 modules 2665.71ms ± 8.61ms 2669.23ms ± 9.03ms +0.13%
bench_startup/Turbopack CSR/1000 modules 1660.35ms ± 4.00ms 1655.29ms ± 4.72ms -0.30%
bench_startup/Turbopack RCC/1000 modules 2547.65ms ± 9.40ms 2548.18ms ± 8.61ms +0.02%
bench_startup/Turbopack RSC/1000 modules 2438.37ms ± 7.61ms 2438.63ms ± 4.78ms +0.01%
bench_startup/Turbopack SSR/1000 modules 2102.02ms ± 7.67ms 2101.86ms ± 5.24ms -0.01%

@kodiakhq kodiakhq bot merged commit a8fa2bb into main Jan 21, 2023
@kodiakhq kodiakhq bot deleted the jrl-jsconfig branch January 21, 2023 03:22
kodiakhq bot pushed a commit to vercel/next.js that referenced this pull request Jan 21, 2023
ijjk added a commit to vercel/next.js that referenced this pull request Jan 26, 2023
Enables the JS CNA template tests for turbopack as well now that we
landed `jsconfig` paths support.

x-ref: vercel/turborepo#3409
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 25, 2024
There doesn't appear to be any issues with just returning a `jsconfig.json` when we're looking for a `tsconfig.json` (provided one didn't exist). Finding this gives a `TsConfigReferenceVc` (used for finding type definitions only) and enables the `paths`/`baseUrl` resolution options.

Fixes WEB-230
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 29, 2024
There doesn't appear to be any issues with just returning a `jsconfig.json` when we're looking for a `tsconfig.json` (provided one didn't exist). Finding this gives a `TsConfigReferenceVc` (used for finding type definitions only) and enables the `paths`/`baseUrl` resolution options.

Fixes WEB-230
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 1, 2024
There doesn't appear to be any issues with just returning a `jsconfig.json` when we're looking for a `tsconfig.json` (provided one didn't exist). Finding this gives a `TsConfigReferenceVc` (used for finding type definitions only) and enables the `paths`/`baseUrl` resolution options.

Fixes WEB-230
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: automerge Kodiak will merge these automatically after checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants