Skip to content

Commit

Permalink
Disable suggesting incremental type checking (#25144)
Browse files Browse the repository at this point in the history
It have performance characteristics that might not work for everyone.
You can still enable it manually.
  • Loading branch information
sokra authored May 14, 2021
1 parent 64a709d commit 84d5766
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
4 changes: 0 additions & 4 deletions packages/next/lib/typescript/writeConfigurationDefaults.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import { promises as fs } from 'fs'
import chalk from 'chalk'
import * as CommentJson from 'next/dist/compiled/comment-json'
import semver from 'next/dist/compiled/semver'
import os from 'os'
import { getTypeScriptConfiguration } from './getTypeScriptConfiguration'

Expand Down Expand Up @@ -29,9 +28,6 @@ function getDesiredCompilerOptions(
strict: { suggested: false },
forceConsistentCasingInFileNames: { suggested: true },
noEmit: { suggested: true },
...(semver.gte(ts.version, '4.3.0-beta')
? { incremental: { suggested: true } }
: undefined),

// These values are required and cannot be changed by the user
// Keep this in sync with the webpack config
Expand Down
6 changes: 0 additions & 6 deletions test/integration/tsconfig-verifier/test/index.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ describe('tsconfig.json verifier', () => {
\\"strict\\": false,
\\"forceConsistentCasingInFileNames\\": true,
\\"noEmit\\": true,
\\"incremental\\": true,
\\"esModuleInterop\\": true,
\\"module\\": \\"esnext\\",
\\"moduleResolution\\": \\"node\\",
Expand Down Expand Up @@ -84,7 +83,6 @@ describe('tsconfig.json verifier', () => {
\\"strict\\": false,
\\"forceConsistentCasingInFileNames\\": true,
\\"noEmit\\": true,
\\"incremental\\": true,
\\"esModuleInterop\\": true,
\\"module\\": \\"esnext\\",
\\"moduleResolution\\": \\"node\\",
Expand Down Expand Up @@ -152,7 +150,6 @@ describe('tsconfig.json verifier', () => {
\\"strict\\": false,
\\"forceConsistentCasingInFileNames\\": true,
\\"noEmit\\": true,
\\"incremental\\": true,
\\"moduleResolution\\": \\"node\\",
\\"resolveJsonModule\\": true,
\\"isolatedModules\\": true,
Expand Down Expand Up @@ -201,7 +198,6 @@ describe('tsconfig.json verifier', () => {
\\"strict\\": false,
\\"forceConsistentCasingInFileNames\\": true,
\\"noEmit\\": true,
\\"incremental\\": true,
\\"moduleResolution\\": \\"node\\",
\\"resolveJsonModule\\": true,
\\"isolatedModules\\": true,
Expand Down Expand Up @@ -247,7 +243,6 @@ describe('tsconfig.json verifier', () => {
\\"strict\\": false,
\\"forceConsistentCasingInFileNames\\": true,
\\"noEmit\\": true,
\\"incremental\\": true,
\\"moduleResolution\\": \\"node\\",
\\"resolveJsonModule\\": true,
\\"isolatedModules\\": true,
Expand Down Expand Up @@ -286,7 +281,6 @@ describe('tsconfig.json verifier', () => {
"strict": false,
"forceConsistentCasingInFileNames": true,
"noEmit": true,
"incremental": true,
"esModuleInterop": true,
"module": "esnext",
"moduleResolution": "node",
Expand Down

0 comments on commit 84d5766

Please sign in to comment.