chore: reorganize CI tests and add timeout (#6779)

* chore: move clang formatter action into the build workflow

* chore: add timeouts to CI tests
This commit is contained in:
Beka Westberg
2023-01-18 09:29:00 -08:00
committed by GitHub
parent ee51c0f41c
commit 856285c6ff
2 changed files with 17 additions and 28 deletions

View File

@@ -10,6 +10,7 @@ permissions:
jobs:
build:
timeout-minutes: 10
runs-on: ${{ matrix.os }}
strategy:
@@ -54,6 +55,7 @@ jobs:
CI: true
lint:
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
@@ -68,3 +70,18 @@ jobs:
- name: Lint
run: npm run lint
clang-formatter:
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: DoozyX/clang-format-lint-action@v0.15
with:
source: 'core'
extensions: 'js,ts'
# This should be as close as possible to the version that the npm
# package supports. This can be found by running:
# npx clang-format --version.
clangFormatVersion: 15

View File

@@ -1,28 +0,0 @@
name: Check clang format
# N.B.: Runs with a read-only repo token. Safe(ish) to check out the
# submitted branch.
on: [pull_request]
permissions:
contents: read
jobs:
clang-formatter:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: DoozyX/clang-format-lint-action@v0.15
with:
source: 'core'
extensions: 'js,ts'
# This should be as close as possible to the version that the npm
# package supports. This can be found by running:
# npx clang-format --version.
clangFormatVersion: 15
# The Report clang format workflow (report_clang_format.yml) will
# run (if required) after this one to post a comment to the PR.
# (Note that the version of that workflow run will be the one on
# the master (default) branch, not the PR target branch.)