Skip to content

Commit f5871eb

Browse files
authored
Merge pull request #7481 from tautschnig/cleanup/syntax-checks
GitHub Actions: cpplint and clang-format checks on PRs only
2 parents 72fb131 + 54034e4 commit f5871eb

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)