chore: migrate from ts-ignore to ts-expect-error (#2103)

Author: SethFalco

lib/xast.js

@@ -44,7 +44,7 @@ export const visitSkip = Symbol();
 export const visit = (node, visitor, parentNode) => {
   const callbacks = visitor[node.type];
   if (callbacks && callbacks.enter) {
-    // @ts-ignore hard to infer
+    // @ts-expect-error hard to infer
     const symbol = callbacks.enter(node, parentNode);
     if (symbol === visitSkip) {
       return;
@@ -66,7 +66,7 @@ export const visit = (node, visitor, parentNode) => {
     }
   }
   if (callbacks && callbacks.exit) {
-    // @ts-ignore hard to infer
+    // @ts-expect-error hard to infer
     callbacks.exit(node, parentNode);
   }
 };

plugins/_path.js

@@ -16,7 +16,7 @@ var prevCtrlPoint;
  * @type {(path: XastElement) => PathDataItem[]}
  */
 export const path2js = (path) => {
-  // @ts-ignore legacy
+  // @ts-expect-error legacy
   if (path.pathJS) return path.pathJS;
   /**
    * @type {PathDataItem[]}
@@ -30,7 +30,7 @@ export const path2js = (path) => {
   if (pathData.length && pathData[0].command == 'm') {
     pathData[0].command = 'M';
   }
-  // @ts-ignore legacy
+  // @ts-expect-error legacy
   path.pathJS = pathData;
   return pathData;
 };
@@ -179,7 +179,7 @@ const convertRelativeToAbsolute = (data) => {
  * @type {(path: XastElement, data: PathDataItem[], params: Js2PathParams) => void}
  */
 export const js2path = function (path, data, params) {
-  // @ts-ignore legacy
+  // @ts-expect-error legacy
   path.pathJS = data;
 
   const pathData = [];
@@ -565,7 +565,7 @@ function gatherPoints(pathData) {
       case 'A':
         if (basePoint != null) {
           // Convert the arc to Bézier curves and use the same approximation
-          // @ts-ignore no idea what's going on here
+          // @ts-expect-error no idea what's going on here
           var curves = a2c.apply(0, basePoint.concat(data));
           for (
             var cData;

plugins/addAttributesToSVGElement.js

@@ -60,14 +60,14 @@ export const fn = (root, params) => {
           for (const attribute of attributes) {
             if (typeof attribute === 'string') {
               if (node.attributes[attribute] == null) {
-                // @ts-ignore disallow explicit nullable attribute value
+                // @ts-expect-error disallow explicit nullable attribute value
                 node.attributes[attribute] = undefined;
               }
             }
             if (typeof attribute === 'object') {
               for (const key of Object.keys(attribute)) {
                 if (node.attributes[key] == null) {
-                  // @ts-ignore disallow explicit nullable attribute value
+                  // @ts-expect-error disallow explicit nullable attribute value
                   node.attributes[key] = attribute[key];
                 }
               }

plugins/cleanupEnableBackground.js

@@ -111,7 +111,7 @@ export const fn = (root) => {
             enableBackgroundDeclaration
           ) {
             const styleValue = csstree.generate(
-              // @ts-ignore
+              // @ts-expect-error
               enableBackgroundDeclaration.data.value,
             );
             const styleCleaned = cleanupValue(
@@ -122,7 +122,7 @@ export const fn = (root) => {
             );
 
             if (styleCleaned) {
-              // @ts-ignore
+              // @ts-expect-error
               enableBackgroundDeclaration.data.value = {
                 type: 'Raw',
                 value: styleCleaned,