Merge pull request #41121 from github/repo-sync

Repo sync

Author: docs-bot

eslint.config.ts

@@ -96,7 +96,6 @@ export default [
       camelcase: 'off', // Many gh apis use underscores, 600+ uses
 
       // Disabled rules to review
-      '@typescript-eslint/ban-ts-comment': 'off', // 50+
       'github/array-foreach': 'off', // 250+
       'no-console': 'off', // 800+
       '@typescript-eslint/no-explicit-any': 'off', // 1000+

src/assets/scripts/validate-asset-images.ts

@@ -17,7 +17,6 @@ import path from 'path'
 import { program } from 'commander'
 import chalk from 'chalk'
 import cheerio from 'cheerio'
-// @ts-ignore see https://github.com/sindresorhus/file-type/issues/652
 import { fileTypeFromFile } from 'file-type'
 import walk from 'walk-sync'
 import isSVG from 'is-svg'

src/content-linter/lib/helpers/get-rules.ts

@@ -4,7 +4,6 @@ import { customConfig } from '@/content-linter/style/github-docs'
 import type { Rule } from '@/content-linter/types'
 
 // Import markdownlint rules - external library without TypeScript declarations
-// @ts-ignore - markdownlint doesn't provide TypeScript declarations
 import markdownlintRules from '../../../../node_modules/markdownlint/lib/rules'
 
 export const customRules: Rule[] = gitHubDocsMarkdownlint.rules

src/content-linter/lib/helpers/print-annotations.ts

@@ -8,7 +8,10 @@
 export function printAnnotationResults(
   // Using 'any' type as results structure is dynamic and comes from various linting tools with different formats
   results: any,
-  { skippableRules = [], skippableFlawProperties = [] } = {},
+  {
+    skippableRules = [],
+    skippableFlawProperties = [],
+  }: { skippableRules?: string[]; skippableFlawProperties?: string[] } = {},
 ) {
   for (const [file, flaws] of Object.entries(results)) {
     // Using 'any' type for flaws as they have varying structures depending on the linting rule

src/content-linter/lib/helpers/utils.ts

@@ -1,4 +1,3 @@
-// @ts-ignore - markdownlint-rule-helpers doesn't provide TypeScript declarations
 import { addError, filterTokens } from 'markdownlint-rule-helpers'
 import matter from '@gr2m/gray-matter'
 

src/content-linter/lib/linting-rules/code-annotation-comment-spacing.ts

@@ -1,4 +1,3 @@
-// @ts-ignore - markdownlint-rule-helpers doesn't provide TypeScript declarations
 import { addError, filterTokens } from 'markdownlint-rule-helpers'
 
 import type { RuleParams, RuleErrorCallback, MarkdownToken } from '@/content-linter/types'

src/content-linter/lib/linting-rules/code-annotations.ts

@@ -1,4 +1,3 @@
-// @ts-ignore - markdownlint-rule-helpers doesn't provide TypeScript declarations
 import { addError, filterTokens } from 'markdownlint-rule-helpers'
 
 import { getFrontmatter } from '@/content-linter/lib/helpers/utils'

src/content-linter/lib/linting-rules/ctas-schema.ts

@@ -1,4 +1,3 @@
-// @ts-ignore - markdownlint-rule-helpers doesn't have TypeScript declarations
 import { addError } from 'markdownlint-rule-helpers'
 import Ajv from 'ajv'
 

src/content-linter/lib/linting-rules/early-access-references.ts

@@ -1,4 +1,3 @@
-// @ts-ignore - markdownlint-rule-helpers doesn't provide TypeScript declarations
 import { addError } from 'markdownlint-rule-helpers'
 import yaml from 'js-yaml'
 

src/content-linter/lib/linting-rules/expired-content.ts

@@ -1,4 +1,3 @@
-// @ts-ignore - markdownlint-rule-helpers doesn't provide TypeScript declarations
 import { addError, newLineRe } from 'markdownlint-rule-helpers'
 
 import type { RuleParams, RuleErrorCallback, MarkdownToken, Rule } from '@/content-linter/types'