From abda3b6c8c6e2688e5fa102dd70422c140bc2966 Mon Sep 17 00:00:00 2001 From: "Acid Chicken (硫酸鶏)" Date: Fri, 7 Apr 2023 20:34:23 +0900 Subject: ci(#10336): use TurboSnap (#10494) * ci(#10336): use TurboSnap * build: fix version * ci(#10336): update build dir * chore(#10336): fire changes * chore: stabilize icon width on `PageHeader` * chore: fire changes * ci: invalid ignore * ci: trace logs * chore: debug * revert: debug This reverts commit 2329165e252640504ef17e97389e0460ada83e70. * chore: do not reuse build dir * build: scripts * ci: tweak * revert: re-revert debug This reverts commit 596ef05d9eabe11c0ec3125fa4a87071d59b9b2f. * chore: detect changes manually * fix: syntax * ci: do not use only-changed * ci: fix command * revert: re-re-revert debug This reverts commit b027170d7504277006ce9abac386df8c38d55fd2. * ci: use build dir * revert: re-re-re-revert debug This reverts commit 529ab126ed63ec10eeeaaee3e9ef581b75e739c9. * ci: fix path * revert: re-re-re-re-revert debug This reverts commit 0b0c0b9ea48d37d617b777e9f602ccf8a2400381. * ci: fix typo * ci: only show story files * revert: re-re-re-re-re-revert debug This reverts commit 9f5b88df3225f23efd72bb099cfebe359af7acdb. * ci: skip when no stories found * ci: use skip * revert: re-re-re-re-re-re-revert debug This reverts commit 0df4bdc30bbf425c8682594d500a6d986517e721. * ci: fix micromatch version * revert: re-re-re-re-re-re-re-revert debug This reverts commit 63063b02bbcb37e85b93afe3f919b185425ddcf0. * revert: re-re-re-re-re-re-re-re-revert debug This reverts commit 01d9669e2a2c758c478bad5379fe19768e2096bc. * chore: pin tabler icon width globally * ci: notify when Chromatic skips * ci: fix endpoint --------- Co-authored-by: syuilo --- .github/workflows/storybook.yml | 40 ++++++++++++++++++++++++++-------------- 1 file changed, 26 insertions(+), 14 deletions(-) (limited to '.github/workflows') diff --git a/.github/workflows/storybook.yml b/.github/workflows/storybook.yml index 9659757c49..1296477faf 100644 --- a/.github/workflows/storybook.yml +++ b/.github/workflows/storybook.yml @@ -2,10 +2,6 @@ name: Storybook on: push: - branches: - - master - - develop - pull_request: branches-ignore: - l10n_develop @@ -13,6 +9,9 @@ jobs: build: runs-on: ubuntu-latest + env: + NODE_OPTIONS: "--max_old_space_size=7168" + steps: - uses: actions/checkout@v3.3.0 with: @@ -36,21 +35,34 @@ jobs: run: pnpm --filter misskey-js build - name: Build storybook run: pnpm --filter frontend build-storybook + - name: Publish to Chromatic + if: github.ref == 'refs/heads/master' + run: pnpm --filter frontend chromatic --exit-once-uploaded -d storybook-static env: - NODE_OPTIONS: "--max_old_space_size=7168" + CHROMATIC_PROJECT_TOKEN: ${{ secrets.CHROMATIC_PROJECT_TOKEN }} - name: Publish to Chromatic + if: github.ref != 'refs/heads/master' id: chromatic - uses: chromaui/action@v1 - with: - exitOnceUploaded: true - projectToken: ${{ secrets.CHROMATIC_PROJECT_TOKEN }} - storybookBuildDir: storybook-static - workingDir: packages/frontend - - name: Compare on Chromatic - if: github.event_name == 'pull_request_target' - run: pnpm --filter frontend chromatic -d storybook-static --exit-once-uploaded --patch-build ${{ github.head_ref }}...${{ github.base_ref }} + run: | + CHROMATIC_PARAMETER="$(node packages/frontend/.storybook/changes.js $(git diff-tree --no-commit-id --name-only -r ${{ github.event.before }} HEAD | xargs))" + if [ "$CHROMATIC_PARAMETER" = " --skip" ]; then + echo "skip=true" >> $GITHUB_OUTPUT + fi + pnpm --filter frontend chromatic --exit-once-uploaded -d storybook-static $(echo "$CHROMATIC_PARAMETER") env: CHROMATIC_PROJECT_TOKEN: ${{ secrets.CHROMATIC_PROJECT_TOKEN }} + - name: Notify that Chromatic will skip testing + uses: actions/github-script@v6.4.0 + if: github.ref != 'refs/heads/master' && steps.chromatic.outputs.skip == 'true' + with: + github-token: ${{ secrets.GITHUB_TOKEN }} + script: | + github.rest.repos.createCommitComment({ + owner: context.repo.owner, + repo: context.repo.repo, + commit_sha: context.sha, + body: 'Chromatic will skip testing but you may still have to [review the changes on Chromatic](https://www.chromatic.com/pullrequests?appId=6428f7d7b962f0b79f97d6e4).' + }) - name: Upload Artifacts uses: actions/upload-artifact@v3 with: -- cgit v1.2.3-freya