Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ci] Properly test mac and linux in GitHub Actions CI #9130

Closed
wants to merge 4 commits into from
Closed

[ci] Properly test mac and linux in GitHub Actions CI #9130

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
671966d
[WIP][ci] Enable tool_test_win
SamChou19815 Mar 18, 2024
014c9b7
Update build_and_test.yml
SamChou19815 Mar 18, 2024
ae113e9
Update build_and_test.yml
SamChou19815 Mar 18, 2024
7477347
Update build_and_test.yml
SamChou19815 Mar 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 37 additions & 17 deletions .github/workflows/build_and_test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -303,6 +303,11 @@ jobs:
FLOW_RUNTESTS_PARALLELISM: 8
steps:
- uses: actions/[email protected]
- uses: actions/[email protected]
with:
name: build_linux_bin
path: bin/linux
- run: chmod +x bin/linux/flow
- name: Run tests
run: ./runtests.sh bin/linux/flow | cat
runtests_macos:
Expand All @@ -314,6 +319,11 @@ jobs:
with:
xcode-version: '13.4'
- uses: actions/[email protected]
- uses: actions/[email protected]
with:
name: build_macos_bin
path: bin/macos
- run: chmod +x bin/macos/flow
- name: Run tests
run: ./runtests.sh bin/macos/flow | cat
tool_test_linux:
Expand All @@ -324,6 +334,11 @@ jobs:
- build_linux
steps:
- uses: actions/[email protected]
- uses: actions/[email protected]
with:
name: build_linux_bin
path: bin/linux
- run: chmod +x bin/linux/flow
- name: Install tool deps from yarn
run: (cd packages/flow-dev-tools && yarn install | cat)
- name: Run tool tests
Expand All @@ -337,27 +352,32 @@ jobs:
with:
xcode-version: '13.4'
- uses: actions/[email protected]
- uses: actions/[email protected]
with:
name: build_macos_bin
path: bin/macos
- run: chmod +x bin/macos/flow
- name: Install tool deps from yarn
run: (cd packages/flow-dev-tools && yarn install | cat)
- name: Run tool tests
run: ./tool test --bin bin/macos/flow | cat
# tool_test_win:
# runs-on: windows-latest
# needs:
# - build_win
# steps:
# - uses: actions/[email protected]
# - name: Install Node LTS
# run: choco install nodejs-lts -y
# - name: Enable Corepack
# run: corepack enable
# - name: Install tool deps from yarn
# run: |-
# cd packages/flow-dev-tools
# yarn install --ignore-scripts --pure-lockfile
# - name: Run tool tests
# run: node packages/flow-dev-tools/bin/tool test --bin bin/win64/flow.exe --parallelism 1
# shell: bash
# runs-on: windows-latest
# needs:
# - build_win
# steps:
# - uses: actions/[email protected]
# - uses: actions/[email protected]
# with:
# name: build_win_bin
# path: bin/win64
# - name: Install tool deps from yarn
# run: |-
# cd packages/flow-dev-tools
# yarn install --ignore-scripts --pure-lockfile
# - name: Run tool tests
# run: node packages/flow-dev-tools/bin/tool test --bin bin/win64/flow.exe --parallelism 1
# shell: bash
ounit_test_linux:
runs-on: ubuntu-latest
container:
Expand Down Expand Up @@ -662,4 +682,4 @@ jobs:
# run: |
# echo "//registry.npmjs.org/:_authToken=${NPM_TOKEN}" > ~/.npmrc
# npm publish ./dist/npm-try-flow-website-js.tgz


Loading