Merge pull request #508 from node-saml/multisaml-tsignore-d-ts

add ts-ignore to generated type definitions for multisaml strategy

Author: gugu

package.json

@@ -40,7 +40,7 @@
     "lint": "eslint --ext .ts src",
     "lint-watch": "onchange 'src/**/*.ts' -- eslint {{file}}",
     "lint:fix": "eslint --ext .ts --fix src",
-    "prepublishOnly": "tsc",
+    "prepare": "tsc",
     "prettier-check": "prettier --config .prettierrc --check .",
     "prettier-format": "prettier --config .prettierrc --write .",
     "prettier-watch": "onchange 'src/**/*.ts' -- prettier --write {{file}}",

src/passport-saml/multiSamlStrategy.ts

@@ -57,7 +57,7 @@ class MultiSamlStrategy extends SamlStrategy {
     });
   }
 
-  // @ts-expect-error typescript disallows changing method signature in a subclass
+  /** @ts-expect-error typescript disallows changing method signature in a subclass */
   generateServiceProviderMetadata(
     req: Request,
     decryptionCert: string | null,