Merge branch 'main' into graphql-schema-update

Author: docubot

.eslintrc.js

@@ -13,7 +13,7 @@ module.exports = {
     babelOptions: { configFile: './.babelrc' },
     sourceType: 'module',
   },
-  ignorePatterns: ['tmp/*'],
+  ignorePatterns: ['tmp/*', '!/.*', '/.next/'],
   rules: {
     'import/no-extraneous-dependencies': ['error', { packageDir: '.' }],
   },

.github/actions-scripts/fr-add-docs-reviewers-requests.js

@@ -134,8 +134,8 @@ async function run() {
   // Exclude existing items going forward.
   // Until we have a way to check from a PR whether the PR is in a project,
   // this is how we (roughly) avoid overwriting PRs that are already on the board
-  let newItemIDs = []
-  let newItemAuthors = []
+  const newItemIDs = []
+  const newItemAuthors = []
   itemIDs.forEach((id, index) => {
     if (!existingItemIDs.includes(id)) {
       newItemIDs.push(id)

.github/actions-scripts/ready-for-docs-review.js

@@ -85,8 +85,8 @@ async function run() {
     // - affected docs sets (not considering changes to data/assets)
     let numFiles = 0
     let numChanges = 0
-    let features = new Set([])
-    const files = data.item.files.nodes.forEach((node) => {
+    const features = new Set([])
+    data.item.files.nodes.forEach((node) => {
       numFiles += 1
       numChanges += node.additions
       numChanges += node.deletions

.github/workflows/browser-test.yml

@@ -6,9 +6,6 @@ name: Browser Tests
 
 on:
   workflow_dispatch:
-  push:
-    branches:
-      - main
   pull_request:
     paths:
       - '**.js'

.github/workflows/code-lint.yml

@@ -18,6 +18,7 @@ on:
       - '**.yaml'
       - '**.yml'
       - '**.scss'
+      - .eslintrc.js
       # In case something like eslint or tsc or prettier upgrades
       - 'package-lock.json'
       # Ultimately, for debugging this workflow itself

.github/workflows/repo-sync.yml

@@ -21,7 +21,7 @@ on:
     - cron: '10,40 * * * *' # every 30 minutes
 
 permissions:
-  contents: read
+  contents: write
   pull-requests: write
 
 jobs:

.github/workflows/staging-deploy-pr.yml

@@ -36,14 +36,6 @@ env:
   BUILD_ACTIONS_RUN_LOG: https://github.com/${{ github.repository }}/actions/runs/${{ github.event.workflow_run.id }}
 
 jobs:
-  debug:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Dump full context for debugging
-        env:
-          GITHUB_CONTEXT: ${{ toJSON(github) }}
-        run: echo "$GITHUB_CONTEXT"
-
   pr-metadata:
     # This is needed because the workflow we depend on
     # (see on.workflow_run.workflows) might be running from pushes on

assets/images/help/codespaces/org-user-permission-settings-outside-collaborators.png

@@ -1,9 +1,8 @@
 import { useRouter } from 'next/router'
-import { useState } from 'react'
 import cx from 'classnames'
 import { ActionList, Heading } from '@primer/components'
 
-import { ChevronDownIcon, ZapIcon, InfoIcon, ShieldLockIcon } from '@primer/octicons-react'
+import { ZapIcon, InfoIcon, ShieldLockIcon } from '@primer/octicons-react'
 import { Callout } from 'components/ui/Callout'
 
 import { Link } from 'components/Link'
@@ -61,9 +60,8 @@ export const ArticlePage = () => {
   } = useArticleContext()
   const { t } = useTranslation('pages')
   const currentPath = router.asPath.split('?')[0]
-  const [isActive, setActive] = useState(-1)
 
-  const renderTocItem = (item: MiniTocItem, index: number) => {
+  const renderTocItem = (item: MiniTocItem) => {
     return (
       <ActionList.Item
         as="li"
@@ -72,19 +70,9 @@ export const ArticlePage = () => {
         sx={{ listStyle: 'none', padding: '2px' }}
       >
         <div className={cx('lh-condensed d-block width-full')}>
-          <div className="d-inline-flex" dangerouslySetInnerHTML={{ __html: item.contents }} />
-          {item.items && item.items.length > 0 && (
-            <button
-              className="color-bg-default border-0 ml-1"
-              onClick={() => setActive(index === isActive ? -1 : index)}
-            >
-              {<ChevronDownIcon />}
-            </button>
-          )}
+          <div dangerouslySetInnerHTML={{ __html: item.contents }} />
           {item.items && item.items.length > 0 ? (
-            <ul className={index === isActive ? 'ml-3' : 'd-none'}>
-              {item.items.map(renderTocItem)}
-            </ul>
+            <ul className="ml-3">{item.items.map(renderTocItem)}</ul>
           ) : null}
         </div>
       </ActionList.Item>
@@ -158,7 +146,7 @@ export const ArticlePage = () => {
                       return {
                         key: title + i,
                         text: title,
-                        renderItem: () => <ul>{renderTocItem(items, i)}</ul>,
+                        renderItem: () => <ul>{renderTocItem(items)}</ul>,
                       }
                     })}
                   />