Skip to content

Commit dcde441

Browse files
mainframevmainframev
committed
chore: update workflow
1 parent 6d57e57 commit dcde441

1 file changed

Lines changed: 153 additions & 122 deletions

File tree

‎.github/workflows/pr.yml‎

Lines changed: 153 additions & 122 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: CI
22
on:
33
push:
44
branches:
5-
- master
5+
- chore/migrate-to-eslint-flat-configs-raw-test
66
pull_request:
77

88
concurrency:
@@ -23,8 +23,8 @@ env:
2323

2424
jobs:
2525
main:
26-
if: ${{ github.repository_owner == 'microsoft' }}
27-
runs-on: macos-14-xlarge
26+
# if: ${{ github.repository_owner == 'microsoft' }}
27+
runs-on: ubuntu-latest
2828
permissions:
2929
contents: 'read'
3030
actions: 'read'
@@ -44,7 +44,7 @@ jobs:
4444
cache: 'yarn'
4545
node-version: '22'
4646

47-
- run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
47+
# - run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
4848

4949
- run: yarn install --frozen-lockfile
5050

@@ -54,125 +54,156 @@ jobs:
5454
yarn nx g @fluentui/workspace-plugin:tsconfig-base-all --verify
5555
yarn nx g @fluentui/workspace-plugin:normalize-package-dependencies --verify
5656
57-
- name: Type-check just.config.ts files
58-
run: |
59-
# following packages need to be build in advance:
60-
# @fluentui/api-docs is used within apps/public-docsite-resources/just.config.ts,
61-
# @fluentui/digest is used within packages/fluentui/perf-test-northstar/just.config.ts, thus it needs to be build in advance
62-
#
63-
# NOTE: we are running this via nx in order to get cache hits later on
64-
yarn nx run-many -t build -p api-docs digest
65-
yarn tsc -p ./tsconfig.just-scripts-configs.json
66-
67-
- name: 'check packages: installed dependencies versions'
68-
run: |
69-
yarn check:installed-dependencies-versions
70-
71-
- name: check formatting
72-
run: |
73-
yarn nx format:check --base origin/master
74-
75-
- name: build, test, lint, test-ssr (affected)
76-
run: |
77-
FLUENT_JEST_WORKER=2 yarn nx affected -t build test lint type-check test-ssr test-integration verify-packaging --exclude react-19-tests-v9 --nxBail
78-
79-
- name: 'Check for unstaged changes'
80-
run: |
81-
git status --porcelain
82-
git diff-index --quiet HEAD -- || exit 1
83-
84-
react-major-versions-integration:
85-
if: ${{ github.repository_owner == 'microsoft' }}
86-
runs-on: macos-14-xlarge
87-
steps:
88-
- uses: actions/checkout@v4
89-
with:
90-
fetch-depth: 0
91-
92-
- name: Derive appropriate SHAs for base and head for `nx affected` commands
93-
uses: nrwl/nx-set-shas@826660b82addbef3abff5fa871492ebad618c9e1 # v4.3.3
94-
with:
95-
main-branch-name: 'master'
57+
# - name: Type-check just.config.ts files
58+
# run: |
59+
# # following packages need to be build in advance:
60+
# # @fluentui/api-docs is used within apps/public-docsite-resources/just.config.ts,
61+
# # @fluentui/digest is used within packages/fluentui/perf-test-northstar/just.config.ts, thus it needs to be build in advance
62+
# #
63+
# # NOTE: we are running this via nx in order to get cache hits later on
64+
# yarn nx run-many -t build -p api-docs digest
65+
# yarn tsc -p ./tsconfig.just-scripts-configs.json
9666

97-
- uses: actions/setup-node@v4
98-
with:
99-
cache: 'yarn'
100-
node-version: '22'
101-
102-
- run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
103-
104-
- run: |
105-
yarn install --frozen-lockfile
106-
yarn rit --react 17 --install-deps
107-
yarn rit --react 18 --install-deps
108-
109-
- name: Verify Cypress installs from RIT temp workspaces (React 17 -> v13, React 18 -> v14)
110-
run: |
111-
"$GITHUB_WORKSPACE"/tmp/rit/react-17/node_modules/.bin/cypress verify
112-
"$GITHUB_WORKSPACE"/tmp/rit/react-18/node_modules/.bin/cypress verify
113-
114-
- name: React Versions Integration Tests (17,18) - E2E
115-
id: e2e
116-
run: |
117-
yarn nx affected -t test-rit--17--e2e,test-rit--18--e2e --exclude='react-19-tests-v9,react-charting,react'
67+
# - name: 'check packages: installed dependencies versions'
68+
# run: |
69+
# yarn check:installed-dependencies-versions
11870

119-
- name: Upload Cypress screenshots if exist
120-
uses: actions/upload-artifact@v4
121-
if: always() && steps.e2e.outcome == 'failure'
122-
with:
123-
name: cypress-screenshots-react-test-rit
124-
path: |
125-
tmp/rit/**/cypress/screenshots/**/*.png
126-
retention-days: 1
71+
# - name: check formatting
72+
# run: |
73+
# yarn nx format:check --base origin/master
12774

128-
- name: React Versions Integration Tests (17,18) - Type-check & Test
75+
- name: build, test, lint, test-ssr (affected)
12976
run: |
130-
FLUENT_JEST_WORKER=2 yarn nx affected -t test-rit--17--type-check,test-rit--18--type-check,test-rit--17--test,test-rit--18--test --exclude='react-19-tests-v9'
131-
132-
e2e:
133-
if: ${{ github.repository_owner == 'microsoft' }}
134-
# TODO: switch to macos once problematic tests are fixed
135-
# https://github.com/microsoft/fluentui/issues/33173
136-
# https://github.com/microsoft/fluentui/issues/33172
137-
runs-on: ubuntu-latest
138-
permissions:
139-
contents: 'read'
140-
actions: 'read'
141-
142-
steps:
143-
- uses: actions/checkout@v4
144-
with:
145-
fetch-depth: 0
146-
147-
- name: Derive appropriate SHAs for base and head for `nx affected` commands
148-
uses: nrwl/nx-set-shas@826660b82addbef3abff5fa871492ebad618c9e1 # v4.3.3
149-
with:
150-
main-branch-name: 'master'
151-
152-
- uses: actions/setup-node@v4
153-
with:
154-
cache: 'yarn'
155-
node-version: '22'
156-
157-
- run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
158-
159-
- run: yarn install --frozen-lockfile
160-
161-
- name: Install Playwright dependencies
162-
run: yarn playwright install --with-deps
163-
164-
- name: Verify Cypress install
165-
run: yarn cypress verify
166-
167-
- name: Cypress/Playwright E2E tests
168-
run: yarn nx affected -t e2e --exclude react-19-tests-v9 --nxBail --parallel 1
169-
170-
- name: Upload Cypress screenshots if exist
171-
uses: actions/upload-artifact@v4
172-
if: failure()
173-
with:
174-
name: cypress-screenshots
175-
path: |
176-
apps/*/cypress/screenshots/**/*.png
177-
packages/**/cypress/screenshots/**/*.png
178-
retention-days: 1
77+
FLUENT_JEST_WORKER=2 yarn nx run-many -t lint --exclude react-19-tests-v9 --nxBail
78+
79+
# - name: 'Check for unstaged changes'
80+
# run: |
81+
# git status --porcelain
82+
# git diff-index --quiet HEAD -- || exit 1
83+
84+
# react-major-versions-integration:
85+
# if: ${{ github.repository_owner == 'microsoft' }}
86+
# runs-on: macos-14-xlarge
87+
# steps:
88+
# - uses: actions/checkout@v4
89+
# with:
90+
# fetch-depth: 0
91+
#
92+
# - name: Derive appropriate SHAs for base and head for `nx affected` commands
93+
# uses: nrwl/nx-set-shas@826660b82addbef3abff5fa871492ebad618c9e1 # v4.3.3
94+
# with:
95+
# main-branch-name: 'master'
96+
#
97+
# - uses: actions/setup-node@v4
98+
# with:
99+
# cache: 'yarn'
100+
# node-version: '22'
101+
#
102+
# - run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
103+
#
104+
# - run: |
105+
# yarn install --frozen-lockfile
106+
# yarn rit --react 17 --install-deps
107+
# yarn rit --react 19 --install-deps
108+
#
109+
# - name: Verify Cypress installs from RIT temp workspaces (React 17 -> v13, React 19 -> v14)
110+
# run: |
111+
# "$GITHUB_WORKSPACE"/tmp/rit/react-17/node_modules/.bin/cypress verify
112+
# "$GITHUB_WORKSPACE"/tmp/rit/react-19/node_modules/.bin/cypress verify
113+
#
114+
# - name: React Versions Integration Tests (17,19) - E2E
115+
# id: e2e
116+
# run: |
117+
# yarn nx affected -t test-rit--17--e2e,test-rit--19--e2e --exclude='react-19-tests-v9,react-charting,react'
118+
#
119+
# - name: Upload Cypress screenshots if exist
120+
# uses: actions/upload-artifact@v4
121+
# if: always() && steps.e2e.outcome == 'failure'
122+
# with:
123+
# name: cypress-screenshots-react-test-rit
124+
# path: |
125+
# tmp/rit/**/cypress/screenshots/**/*.png
126+
# retention-days: 1
127+
#
128+
# - name: React Versions Integration Tests (17,19) - Type-check & Test
129+
# run: |
130+
# FLUENT_JEST_WORKER=2 yarn nx affected -t test-rit--17--type-check,test-rit--19--type-check,test-rit--17--test,test-rit--19--test --exclude='react-19-tests-v9'
131+
132+
# react_19_v9_source_code_typecheck:
133+
# if: ${{ github.repository_owner == 'microsoft' }}
134+
# runs-on: ubuntu-latest
135+
# permissions:
136+
# contents: 'read'
137+
# actions: 'read'
138+
# name: v9 source code type-check against React 19
139+
# steps:
140+
# - uses: actions/checkout@v4
141+
# with:
142+
# fetch-depth: 0
143+
#
144+
# - name: Derive appropriate SHAs for base and head for `nx affected` commands
145+
# uses: nrwl/nx-set-shas@826660b82addbef3abff5fa871492ebad618c9e1 # v4.3.3
146+
# with:
147+
# main-branch-name: 'master'
148+
#
149+
# - uses: actions/setup-node@v4
150+
# with:
151+
# cache: 'yarn'
152+
# node-version: '22'
153+
#
154+
# # - run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
155+
#
156+
# - run: |
157+
# yarn install --frozen-lockfile
158+
# # yarn rit --react 19 --install-deps
159+
#
160+
# - run: |
161+
# yarn nx affected -t test-rit--19--type-check --exclude='*,!react-19-tests-v9'
162+
163+
# e2e:
164+
# if: ${{ github.repository_owner == 'microsoft' }}
165+
# # TODO: switch to macos once problematic tests are fixed
166+
# # https://github.com/microsoft/fluentui/issues/33173
167+
# # https://github.com/microsoft/fluentui/issues/33172
168+
# runs-on: ubuntu-latest
169+
# permissions:
170+
# contents: 'read'
171+
# actions: 'read'
172+
#
173+
# steps:
174+
# - uses: actions/checkout@v4
175+
# with:
176+
# fetch-depth: 0
177+
#
178+
# - name: Derive appropriate SHAs for base and head for `nx affected` commands
179+
# uses: nrwl/nx-set-shas@826660b82addbef3abff5fa871492ebad618c9e1 # v4.3.3
180+
# with:
181+
# main-branch-name: 'master'
182+
#
183+
# - uses: actions/setup-node@v4
184+
# with:
185+
# cache: 'yarn'
186+
# node-version: '22'
187+
#
188+
# - run: echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
189+
#
190+
# - run: yarn install --frozen-lockfile
191+
#
192+
# - name: Install Playwright dependencies
193+
# run: yarn playwright install --with-deps
194+
#
195+
# - name: Verify Cypress install
196+
# run: yarn cypress verify
197+
#
198+
# - name: Cypress/Playwright E2E tests
199+
# run: yarn nx affected -t e2e --exclude react-19-tests-v9 --nxBail --parallel 1
200+
#
201+
# - name: Upload Cypress screenshots if exist
202+
# uses: actions/upload-artifact@v4
203+
# if: failure()
204+
# with:
205+
# name: cypress-screenshots
206+
# path: |
207+
# apps/*/cypress/screenshots/**/*.png
208+
# packages/**/cypress/screenshots/**/*.png
209+
# retention-days: 1

0 commit comments

Comments
 (0)