Skip to content

Commit

Permalink
refactor(ses): update CI.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
leotm committed Aug 30, 2024
1 parent 5dc71f9 commit 7317d10
Showing 1 changed file with 5 additions and 56 deletions.
61 changes: 5 additions & 56 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -231,59 +231,8 @@ jobs:
- name: Run yarn test262
run: exit 0 # TODO remove test262 from required tests for CI

platform-compatibility-test:
name: platform-compatibility-test

# begin macro

runs-on: ${{ matrix.platform }}
strategy:
fail-fast: false
matrix:
node-version: [18.x, 20.x]
platform: [ubuntu-latest]

steps:
- name: Checkout
uses: actions/checkout@v3

# without this, setup-node errors on mismatched yarn versions
- run: corepack enable

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
cache: yarn

- name: Echo node version
run: node --version

- name: Install dependencies
run: yarn install --immutable

# end macro

- name: 'build'
run: yarn run build

# fails under Node v12
- run: corepack disable

- name: 'switch to node v12'
uses: actions/setup-node@v3
with:
node-version: '12.x'

- name: Echo node version
run: node --version

- name: Run test:platform-compatibility
# npm b/c Yarn 4 doesn't work in Node 12
run: cd packages/ses && npm run test:platform-compatibility

hermes-test:
name: hermes-test
test-hermes:
name: test-hermes

# begin macro

Expand Down Expand Up @@ -315,10 +264,10 @@ jobs:

# end macro

- name: 'build'
run: yarn run build
- name: Run yarn build
run: yarn build

- name: Run SES smoke test on Hermes
- name: Run SES/Hermes smoke test
run: cd packages/ses && yarn test:hermes

viable-release:
Expand Down

0 comments on commit 7317d10

Please sign in to comment.