Skip to content

[ci] Move docs building to github actions. NFC #24226

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
66 changes: 0 additions & 66 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -433,15 +433,6 @@ commands:
- upload-test-results

jobs:
build-docs:
executor: focal
steps:
- checkout
- pip-install
- run: tools/maint/update_settings_docs.py --check
- run: make -C site text
- run: tools/maint/check_emcc_help_text.py
- run: make -C site html
ruff:
executor: focal
steps:
Expand Down Expand Up @@ -1088,60 +1079,3 @@ jobs:
title: "crossplatform tests"
test_targets: "--crossplatform-only"
- upload-test-results

workflows:
build-test:
jobs:
- ruff
- mypy
- eslint
- build-docs
- build-linux
- test-sanity:
requires:
- build-linux
- test-posixtest:
requires:
- build-linux
- test-core0:
requires:
- build-linux
- test-core2:
requires:
- build-linux
- test-core3:
requires:
- build-linux
- test-wasm64-4gb:
requires:
- build-linux
- test-wasm2js1:
requires:
- build-linux
- test-other:
requires:
- build-linux
- test-browser-chrome
- test-browser-chrome-2gb:
requires:
- build-linux
- test-browser-chrome-wasm64:
requires:
- build-linux
- test-browser-chrome-wasm64-4gb:
requires:
- build-linux
- test-browser-firefox:
requires:
- build-linux
- test-browser-firefox-wasm64
- test-sockets-chrome:
requires:
- build-linux
- test-jsc
- test-spidermonkey
- test-node-compat
- test-windows
- test-mac-arm64:
requires:
- build-linux
19 changes: 17 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,6 @@ jobs:
fetch-depth: 0 # We want access to other branches, specifically `main`
- name: pip install
run: |
which python3
python3 --version
python3 -m pip install -r requirements-dev.txt
- name: Install emsdk
run: |
Expand Down Expand Up @@ -87,3 +85,20 @@ jobs:
echo "-- This failure is only a warning and can be ignored"
exit 1
fi

build-docs:
name: Build Docs
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608 # v4.1.0
- name: pip install
run: |
python3 -m pip install -r requirements-dev.txt
- run: tools/maint/update_settings_docs.py --check
- run: make -C site text
- run: tools/maint/check_emcc_help_text.py
- run: make -C site html
- uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.5
with:
name: docs
path: site/build/html/