Add strict to tsconfig.json

Author: wooorm

lib/index.js

@@ -48,7 +48,7 @@ export const x =
 
       if (name === undefined || name === null) {
         node = {type: 'root', children: []}
-        // @ts-ignore Root builder doesn’t accept attributes.
+        // @ts-expect-error Root builder doesn’t accept attributes.
         children.unshift(attributes)
       } else if (typeof name === 'string') {
         node = {type: 'element', name, attributes: {}, children: []}
@@ -65,7 +65,7 @@ export const x =
               (typeof attributes[key] !== 'number' ||
                 !Number.isNaN(attributes[key]))
             ) {
-              // @ts-ignore Pretty sure we just set it.
+              // @ts-expect-error Pretty sure we just set it.
               node.attributes[key] = String(attributes[key])
             }
           }

script/generate-jsx.js

@@ -12,10 +12,10 @@ fs.writeFileSync(
   path.join('test', 'jsx-build-jsx-classic.js'),
   generate(
     buildJsx(
-      // @ts-ignore Acorn nodes are assignable to ESTree nodes.
+      // @ts-expect-error Acorn nodes are assignable to ESTree nodes.
       Parser.extend(acornJsx()).parse(
         doc.replace(/'name'/, "'jsx (estree-util-build-jsx, classic)'"),
-        // @ts-ignore Hush, `2021` is fine.
+        // @ts-expect-error Hush, `2021` is fine.
         {sourceType: 'module', ecmaVersion: 2021}
       ),
       {pragma: 'x', pragmaFrag: 'null'}
@@ -27,10 +27,10 @@ fs.writeFileSync(
   path.join('test', 'jsx-build-jsx-automatic.js'),
   generate(
     buildJsx(
-      // @ts-ignore Acorn nodes are assignable to ESTree nodes.
+      // @ts-expect-error Acorn nodes are assignable to ESTree nodes.
       Parser.extend(acornJsx()).parse(
         doc.replace(/'name'/, "'jsx (estree-util-build-jsx, automatic)'"),
-        // @ts-ignore Hush, `2021` is fine.
+        // @ts-expect-error Hush, `2021` is fine.
         {sourceType: 'module', ecmaVersion: 2021}
       ),
       {runtime: 'automatic', importSource: '.'}
@@ -40,7 +40,7 @@ fs.writeFileSync(
 
 fs.writeFileSync(
   path.join('test', 'jsx-babel-classic.js'),
-  // @ts-ignore Result always given.
+  // @ts-expect-error Result always given.
   babel.transform(doc.replace(/'name'/, "'jsx (babel, classic)'"), {
     plugins: [
       ['@babel/plugin-transform-react-jsx', {pragma: 'x', pragmaFrag: 'null'}]
@@ -50,7 +50,7 @@ fs.writeFileSync(
 
 fs.writeFileSync(
   path.join('test', 'jsx-babel-automatic.js'),
-  // @ts-ignore Result always given.
+  // @ts-expect-error Result always given.
   babel
     .transformSync(doc.replace(/'name'/, "'jsx (babel, automatic)'"), {
       plugins: [

test/core.js

@@ -12,7 +12,7 @@ test('xastscript', (t) => {
 
   t.throws(
     () => {
-      // @ts-ignore runtime.
+      // @ts-expect-error runtime.
       x(1)
     },
     /Expected element name, got `1`/,
@@ -69,7 +69,7 @@ test('xastscript', (t) => {
   )
 
   t.deepEqual(
-    // @ts-ignore Deeply nested children are not typed.
+    // @ts-expect-error Deeply nested children are not typed.
     x('y', {}, [[[x('a')]], [[[[x('b')]], x('c')]]]),
     {
       type: 'element',
@@ -120,7 +120,7 @@ test('xastscript', (t) => {
 
   t.throws(
     () => {
-      // @ts-ignore runtime.
+      // @ts-expect-error runtime.
       x('y', {}, {})
     },
     /Expected node, nodes, string, got `\[object Object]`/,

tsconfig.json

@@ -19,6 +19,7 @@
     "emitDeclarationOnly": true,
     "allowSyntheticDefaultImports": true,
     "skipLibCheck": true,
-    "strictNullChecks": true
+    "strictNullChecks": true,
+    "strict": true
   }
 }