Merge pull request #43279 from github/repo-sync

Repo sync

Author: docs-bot

eslint.config.ts

@@ -193,7 +193,6 @@ export default [
       'src/article-api/scripts/generate-api-docs.ts',
       'src/article-api/transformers/audit-logs-transformer.ts',
       'src/article-api/transformers/rest-transformer.ts',
-      'src/automated-pipelines/tests/rendering.ts',
       'src/codeql-cli/scripts/convert-markdown-for-docs.ts',
       'src/content-linter/lib/helpers/get-lintable-yml.ts',
       'src/content-linter/lib/helpers/print-annotations.ts',
@@ -229,16 +228,11 @@ export default [
       'src/content-render/unified/index.ts',
       'src/content-render/unified/module-types.d.ts',
       'src/content-render/unified/rewrite-local-links.ts',
-      'src/data-directory/lib/data-directory.ts',
-      'src/data-directory/lib/data-schemas/learning-tracks.ts',
       'src/data-directory/lib/get-data.ts',
-      'src/data-directory/scripts/find-orphaned-features/find.ts',
       'src/early-access/scripts/migrate-early-access-product.ts',
       'src/early-access/scripts/what-docs-early-access-branch.ts',
       'src/fixtures/tests/categories-and-subcategory.ts',
       'src/fixtures/tests/guides.ts',
-      'src/fixtures/tests/liquid.ts',
-      'src/fixtures/tests/markdown.ts',
       'src/fixtures/tests/translations.ts',
       'src/frame/components/context/ArticleContext.tsx',
       'src/frame/components/context/CategoryLandingContext.tsx',
@@ -258,13 +252,10 @@ export default [
       'src/github-apps/lib/index.ts',
       'src/graphql/lib/index.ts',
       'src/graphql/pages/reference.tsx',
-      'src/graphql/scripts/build-changelog.ts',
-      'src/graphql/scripts/utils/process-previews.ts',
       'src/graphql/scripts/utils/process-schemas.ts',
       'src/graphql/scripts/utils/schema-helpers.ts',
       'src/graphql/tests/validate-schema.ts',
       'src/journeys/lib/journey-path-resolver.ts',
-      'src/journeys/middleware/journey-track.ts',
       'src/landings/components/CookBookFilter.tsx',
       'src/landings/components/ProductGuidesContext.tsx',
       'src/landings/components/ProductLandingContext.tsx',
@@ -274,13 +265,10 @@ export default [
       'src/languages/lib/correct-translation-content.ts',
       'src/languages/lib/render-with-fallback.ts',
       'src/languages/lib/translation-utils.ts',
-      'src/learning-track/lib/process-learning-tracks.ts',
       'src/links/lib/update-internal-links.ts',
       'src/links/scripts/check-github-github-links.ts',
       'src/links/scripts/update-internal-links.ts',
-      'src/links/tests/extract-links.ts',
       'src/pages/_error.tsx',
-      'src/products/tests/get-product-groups.ts',
       'src/redirects/middleware/handle-redirects.ts',
       'src/rest/components/get-rest-code-samples.ts',
       'src/rest/lib/index.ts',
@@ -319,13 +307,8 @@ export default [
       'src/types/markdownlint-rule-helpers.d.ts',
       'src/types/markdownlint-rule-search-replace.d.ts',
       'src/types/primer__octicons.d.ts',
-      'src/versions/scripts/update-versioning-in-files.ts',
       'src/versions/scripts/use-short-versions.ts',
       'src/webhooks/lib/index.ts',
-      'src/webhooks/scripts/webhook.ts',
-      'src/workflows/content-changes-table-comment.ts',
-      'src/workflows/fr-add-docs-reviewers-requests.ts',
-      'src/workflows/git-utils.ts',
       'src/workflows/projects.ts',
       'src/workflows/tests/actions-workflows.ts',
     ],

src/automated-pipelines/tests/rendering.ts

@@ -12,8 +12,8 @@ type Page = {
   fullPath: string
   permalinks: Array<{ href: string }>
   versions: {
-    feature?: any
-    [key: string]: any
+    feature?: string | string[]
+    [key: string]: string | string[] | undefined
   }
 }
 

src/data-directory/lib/data-directory.ts

@@ -14,7 +14,7 @@ interface DataDirectoryOptions {
 }
 
 interface DataDirectoryResult {
-  [key: string]: any
+  [key: string]: unknown
 }
 
 export default function dataDirectory(

src/data-directory/lib/data-schemas/learning-tracks.ts

@@ -2,7 +2,7 @@ import { schema } from '@/frame/lib/frontmatter'
 
 // Some learning tracks have `versions` blocks that match `versions` frontmatter,
 // so we can import that part of the FM schema.
-const versionsProps = Object.assign({}, (schema.properties as any).versions)
+const versionsProps = Object.assign({}, schema.properties.versions)
 
 // `versions` are not required in learning tracks the way they are in FM.
 delete versionsProps.required

src/data-directory/scripts/find-orphaned-features/find.ts

@@ -317,7 +317,7 @@ function escapeRegex(string: string) {
   return string.replace(/[/\-\\^$*+?.()|[\]{}]/g, '\\$&')
 }
 
-function isFloat(x: any) {
+function isFloat(x: string | number) {
   return !!(Number(x) + 1)
 }
 strictEqual(isFloat('1.2'), true)

src/data-directory/tests/index.ts

@@ -24,7 +24,7 @@ describe('data-directory', () => {
       return content.replace('markdown', 'MARKDOWN')
     }
     const data = dataDirectory(fixturesDir, { preprocess })
-    expect(data.nested.baz).toBe('I am MARKDOWN!')
+    expect((data.nested as Record<string, unknown>).baz).toBe('I am MARKDOWN!')
   })
 
   test('option: extensions array', async () => {

src/fixtures/tests/liquid.ts

@@ -131,10 +131,10 @@ describe('rowheaders', () => {
     // `scope` attribute.
     // See "Scope attribute should be used correctly on tables"
     // https://dequeuniversity.com/rules/axe/4.1/scope-attr-valid?application=RuleDescription
-    $('thead th', firstTable).each((i: number, element: any) => {
+    $('thead th', firstTable).each((i, element) => {
       expect($(element).attr('scope')).toBe('col')
     })
-    $('tbody th', firstTable).each((i: number, element: any) => {
+    $('tbody th', firstTable).each((i, element) => {
       expect($(element).attr('scope')).toBe('row')
     })
     // The 5 here is the other `expect(...)` that happens before these
@@ -222,7 +222,7 @@ describe('misc Liquid', () => {
     const links = $(`#article-contents a[href="${url}"]`)
     expect(links.length).toBe(2)
     const texts = links
-      .map((i: number, element: any) => {
+      .map((i, element) => {
         return $(element).text()
       })
       .get()
@@ -293,7 +293,7 @@ describe('data tag', () => {
     // But because `{% data reusables.injectables.paragraphs %}` is
     // inserted with some indentation, that's replicated on every line.
     const li = $('#article-contents li')
-      .filter((_: number, element: any) => {
+      .filter((_, element) => {
         return $(element).text().trim().startsWith('Point 1')
       })
       .eq(0)

src/fixtures/tests/markdown.ts

@@ -23,11 +23,11 @@ describe('alerts', () => {
     const svgs = $('svg', alerts)
     expect(svgs.length).toBe(5)
     const titles = $('.ghd-alert-title', alerts)
-      .map((_: number, el: any) => $(el).text())
+      .map((_, el) => $(el).text())
       .get()
     expect(titles).toEqual(['Tip', 'Note', 'Important', 'Warning', 'Caution'])
     const bodies = $('p:nth-child(2)', alerts)
-      .map((_: number, el: any) => $(el).text())
+      .map((_, el) => $(el).text())
       .get()
       .map((s: string) => s.trim())
     expect(bodies).toEqual([

src/graphql/scripts/build-changelog.ts

@@ -57,6 +57,8 @@ interface IgnoredChangesSummary {
   types: IgnoredChangeType[]
 }
 
+let lastIgnoredChanges: Change[] = []
+
 /**
  * Tag `changelogEntry` with `date: YYYY-mm-dd`, then prepend it to the JSON
  * structure written to `targetPath`. (`changelogEntry` and that file are modified in place.)
@@ -89,9 +91,8 @@ export async function createChangelogEntry(
   newUpcomingChanges: UpcomingChange[],
 ): Promise<ChangelogEntry | null> {
   // Create schema objects out of the strings
-  // Using 'as any' because loadSchema accepts string schema directly without requiring loaders
-  const oldSchema = await loadSchema(oldSchemaString, {} as any)
-  const newSchema = await loadSchema(newSchemaString, {} as any)
+  const oldSchema = await loadSchema(oldSchemaString, { loaders: [] })
+  const newSchema = await loadSchema(newSchemaString, { loaders: [] })
 
   // Generate changes between the two schemas
   const changes = await diff(oldSchema, newSchema)
@@ -122,7 +123,7 @@ export async function createChangelogEntry(
   }
 
   // Store ignored changes for potential workflow outputs
-  ;(createChangelogEntry as any).lastIgnoredChanges = ignoredChanges
+  lastIgnoredChanges = ignoredChanges
 
   const { schemaChangesToReport, previewChangesToReport } = segmentPreviewChanges(
     changesToReport,
@@ -333,7 +334,7 @@ const CHANGES_TO_REPORT = [
  * Get the ignored change types from the last changelog entry creation
  */
 export function getLastIgnoredChanges(): Change[] {
-  return (createChangelogEntry as any).lastIgnoredChanges || []
+  return lastIgnoredChanges
 }
 
 /**

src/graphql/scripts/sync.ts

@@ -31,10 +31,12 @@ interface IgnoredChange {
 
 interface RawPreview {
   title: string
+  description?: string
   toggled_on: string[]
   toggled_by: string
   announcement?: unknown
   updates?: unknown
+  owning_teams?: string[]
 }
 
 interface UpcomingChangeEntry {