Skip to content

Commit

Permalink
Merge branch 'main' into 4315-select-panel-state
Browse files Browse the repository at this point in the history
  • Loading branch information
ipc103 authored May 22, 2024
2 parents f916ed6 + 9b63299 commit b4253e9
Show file tree
Hide file tree
Showing 2,675 changed files with 33,449 additions and 19,853 deletions.
2 changes: 1 addition & 1 deletion .changeset/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,5 @@
"access": "public",
"baseBranch": "main",
"updateInternalDependencies": "patch",
"ignore": ["docs", "example-*"]
"ignore": ["docs", "example-*", "codesandbox"]
}
5 changes: 0 additions & 5 deletions .changeset/fair-trainers-battle.md

This file was deleted.

5 changes: 5 additions & 0 deletions .changeset/few-weeks-serve.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@primer/react': patch
---

Add `openInFocus` prop (default: true) to `AutocompleteInput`
5 changes: 0 additions & 5 deletions .changeset/khaki-schools-lay.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/nervous-dogs-change.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/old-peas-cross.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/warm-steaks-fry.md

This file was deleted.

6 changes: 0 additions & 6 deletions .changeset/young-bikes-kneel.md

This file was deleted.

7 changes: 6 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,11 @@ module.exports = {
camelcase: [
'error',
{
allow: ['dark_dimmed'],
allow: [
'dark_dimmed',
// Allow feature flag names that start with `primer_react_`
'^primer_react_',
],
},
],
'primer-react/no-deprecated-colors': ['warn', {checkAllStrings: true}],
Expand Down Expand Up @@ -206,6 +210,7 @@ module.exports = {
'no-unused-vars': 'off',
'prettier/prettier': 'off',
'react/jsx-no-undef': 'off',
'react/no-unescaped-entities': 'off',
'primer-react/direct-slot-children': 'off',
},
},
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/accessibility-alt-text-bot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,4 +21,4 @@ jobs:
if: ${{ github.event.issue || github.event.pull_request || github.event.discussion }}
steps:
- name: Get action 'github/accessibility-alt-text-bot'
uses: github/accessibility-alt-text-bot@v1.4.0
uses: github/accessibility-alt-text-bot@v1.5.0
19 changes: 10 additions & 9 deletions .github/workflows/assign_release_conductor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ jobs:
- uses: actions/setup-node@v4
with:
node-version: 20
- run: npm i -g npm@^10.5.1
- run: npm ci
- uses: ./.github/actions/pagerduty
id: pagerduty
Expand Down Expand Up @@ -65,15 +66,6 @@ jobs:
return user.login === PREV_RELEASE_CONDUCTOR;
});
if (hasPreviousReviewer) {
await github.rest.pulls.removeRequestedReviewers({
owner: context.repo.owner,
repo: context.repo.repo,
pull_number: PR_NUMBER,
reviewers: [PREV_RELEASE_CONDUCTOR],
});
}
// Add the current release conductor as an assignee if they are not currently assigned
const hasAssignee = pull.assignees.find((assignee) => {
return assignee.login === RELEASE_CONDUCTOR;
Expand All @@ -100,3 +92,12 @@ jobs:
reviewers: [RELEASE_CONDUCTOR]
})
}
if (hasPreviousReviewer) {
await github.rest.pulls.removeRequestedReviewers({
owner: context.repo.owner,
repo: context.repo.repo,
pull_number: PR_NUMBER,
reviewers: [PREV_RELEASE_CONDUCTOR],
});
}
51 changes: 27 additions & 24 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Check for unformatted files
Expand All @@ -37,19 +38,16 @@ jobs:
steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 20
cache: 'npm'

- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci

- name: Lint JavaScript
run: npm run lint

- name: Lint markdown
run: npm run lint:md

Expand All @@ -58,19 +56,16 @@ jobs:
steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 20
cache: 'npm'

- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci

- name: Build
run: npm run build

- name: Test
run: npm run test -- --coverage

Expand All @@ -79,19 +74,16 @@ jobs:
steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 20
cache: 'npm'

- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci

- name: Build project
run: npm run build

- name: Type check
run: npm run type-check

Expand All @@ -105,6 +97,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build
Expand All @@ -123,6 +116,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build storybook
Expand All @@ -137,7 +131,7 @@ jobs:
echo "pid=$pid" >> $GITHUB_OUTPUT
sleep 5
- name: Run VRT
uses: docker://mcr.microsoft.com/playwright:v1.37.0-jammy
uses: docker://mcr.microsoft.com/playwright:v1.43.0-jammy
env:
STORYBOOK_URL: 'http://172.17.0.1:6006'
with:
Expand All @@ -146,10 +140,11 @@ jobs:
run: kill ${{ steps.storybook.outputs.pid }}
- name: Upload report
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: vrt
name: vrt-${{ matrix.shard }}
path: blob-report
retention-days: 1

vrt:
if: ${{ always() }}
Expand All @@ -162,17 +157,19 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: install dependencies
run: npm ci
- name: download all reports
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: vrt
path: all-blob-reports
pattern: vrt-*
merge-multiple: true
- name: merge all reports
run: npx playwright merge-reports --reporter html ./all-blob-reports
- name: Upload report
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: vrt
path: playwright-report
Expand All @@ -193,6 +190,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build storybook
Expand All @@ -207,7 +205,7 @@ jobs:
echo "pid=$pid" >> $GITHUB_OUTPUT
sleep 5
- name: Run AAT
uses: docker://mcr.microsoft.com/playwright:v1.37.0-jammy
uses: docker://mcr.microsoft.com/playwright:v1.43.0-jammy
env:
STORYBOOK_URL: 'http://172.17.0.1:6006'
with:
Expand All @@ -216,10 +214,11 @@ jobs:
run: kill ${{ steps.storybook.outputs.pid }}
- name: Upload report
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: axe
name: axe-${{ matrix.shard }}
path: blob-report
retention-days: 1

aat:
if: ${{ always() }}
Expand All @@ -232,17 +231,19 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: install dependencies
run: npm ci
- name: download all reports
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: axe
path: all-blob-reports
pattern: axe-*
merge-multiple: true
- name: merge all reports
run: npx playwright merge-reports --reporter html ./all-blob-reports
- name: Upload report
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: axe
path: playwright-report
Expand All @@ -260,6 +261,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build components.json
Expand All @@ -275,6 +277,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/consumer_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ jobs:
with:
node-version: 20
cache: npm
- run: npm i -g npm@^10.5.1
- name: Remove "prepare" script
run: npm pkg delete scripts.prepare
- name: Install dependencies
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/deploy_preview.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build docs preview
Expand Down Expand Up @@ -63,15 +64,15 @@ jobs:
environment_url: '${{ needs.deploy-preview.outputs.deployment_url }}/storybook'
- name: Update storybook deployment status (success)
if: success()
uses: chrnorm/deployment-status@v2.0.1
uses: chrnorm/deployment-status@v2.0.3
with:
token: ${{ secrets.GITHUB_TOKEN }}
environment-url: '${{ needs.deploy-preview.outputs.deployment_url }}/storybook'
state: 'success'
deployment-id: ${{ steps.storybook.outputs.deployment_id }}
- name: Update storybook deployment status (failure)
if: failure()
uses: chrnorm/deployment-status@v2.0.1
uses: chrnorm/deployment-status@v2.0.3
with:
token: ${{ secrets.GITHUB_TOKEN }}
state: 'failure'
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/deploy_preview_forks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ jobs:
with:
node-version: 20
cache: 'npm'
- run: npm i -g npm@^10.5.1
- name: Install dependencies
run: npm ci
- name: Build docs preview
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/deploy_production.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
name: Production
needs: [guard]
if: ${{ needs.guard.outputs.should_deploy == 'true' }}
uses: primer/.github/.github/workflows/deploy.yml@bb0b18fb589038a93a49a24624be0559c01ed0b9
uses: primer/.github/.github/workflows/deploy.yml@5d641c4a200cecaddcb067822ea54dd17ae0a821
secrets:
gh_token: ${{ secrets.GITHUB_TOKEN }}
with:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/release-schedule.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: 20
- run: npm i -g npm@^10.5.1
- name: Install packages for github-script
run: npm i date-fns
- name: Create Release Issue
Expand Down
Loading

0 comments on commit b4253e9

Please sign in to comment.