Skip to content

Commit 54034e4

Browse files
committed
GitHub Actions: cpplint and clang-format checks on PRs only
With 2964edf various checks are now also being run after the merge into develop (so as to make sure that GitHub's cache is adequately populated). The syntactic checks of the PR-specific diff neither work nor make sense upon push. This commit moves them into a YAML file of their own and limits their execution to pull requests only. No code changes.
1 parent 5cabd3b commit 54034e4

File tree

2 files changed

+49
-44
lines changed

2 files changed

+49
-44
lines changed

.github/workflows/pull-request-checks.yaml

Lines changed: 0 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -593,50 +593,6 @@ jobs:
593593
- name: Run CBMC regression tests
594594
run: make CXX=clcache BUILD_ENV=MSVC -C regression test
595595

596-
# This job takes approximately 1 minute
597-
check-clang-format:
598-
runs-on: ubuntu-20.04
599-
steps:
600-
- uses: actions/checkout@v3
601-
with:
602-
submodules: recursive
603-
fetch-depth: 0
604-
- name: Fetch dependencies
605-
env:
606-
# This is needed in addition to -yq to prevent apt-get from asking for
607-
# user input
608-
DEBIAN_FRONTEND: noninteractive
609-
run: |
610-
sudo apt-get update
611-
sudo apt-get install --no-install-recommends -yq clang-format-11
612-
- name: Check updated lines of code match clang-format-11 style
613-
env:
614-
BASE_BRANCH: ${{ github.base_ref }}
615-
MERGE_BRANCH: ${{ github.ref }}
616-
run: ./.github/workflows/pull-request-check-clang-format.sh
617-
618-
# This job takes approximately 1 minute
619-
check-cpplint:
620-
runs-on: ubuntu-20.04
621-
steps:
622-
- uses: actions/checkout@v3
623-
with:
624-
submodules: recursive
625-
fetch-depth: 0
626-
- name: Fetch dependencies
627-
env:
628-
# This is needed in addition to -yq to prevent apt-get from asking for
629-
# user input
630-
DEBIAN_FRONTEND: noninteractive
631-
run: |
632-
sudo apt-get update
633-
sudo apt-get install --no-install-recommends -yq python3-unidiff
634-
- name: Check updated lines of code meet linting standards
635-
env:
636-
BASE_BRANCH: ${{ github.base_ref }}
637-
MERGE_BRANCH: ${{ github.ref }}
638-
run: ./.github/workflows/pull-request-check-cpplint.sh
639-
640596
# This job takes approximately 32 minutes
641597
windows-msi-package:
642598
runs-on: windows-2019

.github/workflows/syntax-checks.yaml

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
name: Syntactic checks
2+
on:
3+
pull_request:
4+
branches: [ develop ]
5+
6+
jobs:
7+
# This job takes approximately 1 minute
8+
check-clang-format:
9+
runs-on: ubuntu-20.04
10+
steps:
11+
- uses: actions/checkout@v3
12+
with:
13+
submodules: recursive
14+
fetch-depth: 0
15+
- name: Fetch dependencies
16+
env:
17+
# This is needed in addition to -yq to prevent apt-get from asking for
18+
# user input
19+
DEBIAN_FRONTEND: noninteractive
20+
run: |
21+
sudo apt-get update
22+
sudo apt-get install --no-install-recommends -yq clang-format-11
23+
- name: Check updated lines of code match clang-format-11 style
24+
env:
25+
BASE_BRANCH: ${{ github.base_ref }}
26+
MERGE_BRANCH: ${{ github.ref }}
27+
run: ./.github/workflows/pull-request-check-clang-format.sh
28+
29+
# This job takes approximately 1 minute
30+
check-cpplint:
31+
runs-on: ubuntu-20.04
32+
steps:
33+
- uses: actions/checkout@v3
34+
with:
35+
submodules: recursive
36+
fetch-depth: 0
37+
- name: Fetch dependencies
38+
env:
39+
# This is needed in addition to -yq to prevent apt-get from asking for
40+
# user input
41+
DEBIAN_FRONTEND: noninteractive
42+
run: |
43+
sudo apt-get update
44+
sudo apt-get install --no-install-recommends -yq python3-unidiff
45+
- name: Check updated lines of code meet linting standards
46+
env:
47+
BASE_BRANCH: ${{ github.base_ref }}
48+
MERGE_BRANCH: ${{ github.ref }}
49+
run: ./.github/workflows/pull-request-check-cpplint.sh

0 commit comments

Comments
 (0)