diff options
| author | Sam Clegg <sbc@chromium.org> | 2021-03-18 16:59:21 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-18 16:59:21 -0700 |
| commit | 2c14fc3c11052e1db1bac3beb8193b9f05e45412 (patch) | |
| tree | d3d3fd40596c61e91f88939b4ccdacdda77fb51f | |
| parent | 5575c13b1e8faef31e11ee0f009cd0f5d78b3e65 (diff) | |
| download | binaryen-2c14fc3c11052e1db1bac3beb8193b9f05e45412.tar.gz binaryen-2c14fc3c11052e1db1bac3beb8193b9f05e45412.tar.bz2 binaryen-2c14fc3c11052e1db1bac3beb8193b9f05e45412.zip | |
Perform clang-tidy/clang-format linting only on PRs (#3708)
These steps diff between a PR and upstream which doesn't
really make sense for normal CI runs.
| -rw-r--r-- | .github/workflows/ci.yml | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 9295f6b67..41bbb031e 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -13,6 +13,7 @@ jobs: lint: name: lint + if: ${{ github.event_name == 'pull_request' }} runs-on: ubuntu-latest steps: - uses: actions/setup-python@v1 @@ -32,8 +33,6 @@ jobs: # clang-tidy requires compile_commands.json generated by cmake cmake . -DCMAKE_EXPORT_COMPILE_COMMANDS=ON ./scripts/clang-tidy-diff.sh - - name: gen-s-parser - run: ./scripts/gen-s-parser.py | diff src/gen-s-parser.inc - build: name: build @@ -50,6 +49,10 @@ jobs: - name: install Python dev dependencies run: pip3 install -r requirements-dev.txt + - name: gen-s-parser + run: ./scripts/gen-s-parser.py | diff src/gen-s-parser.inc - + if: matrix.os == 'ubuntu-latest' + - name: install ninja (linux) run: sudo apt-get install ninja-build if: matrix.os == 'ubuntu-latest' |