diff options
author | rusty-snake <41237666+rusty-snake@users.noreply.github.com> | 2020-11-29 14:04:22 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-29 14:04:22 +0000 |
commit | 4b0b7ec216ec1a1f337a3a37b2c514bcd6842629 (patch) | |
tree | bb9c139f63e47dc2d6cd8c30f4369f5f7ce19128 /.github/workflows | |
parent | Fix building C# projects in Godot (#3778) (diff) | |
download | firejail-4b0b7ec216ec1a1f337a3a37b2c514bcd6842629.tar.gz firejail-4b0b7ec216ec1a1f337a3a37b2c514bcd6842629.tar.zst firejail-4b0b7ec216ec1a1f337a3a37b2c514bcd6842629.zip |
Update build.yml (#3779)
* Update build.yml
Currently we run all actions for all commits. This is not resource friendly. Let's tweak this a bit.
This commit adds support for "skip ci" tags (i.e. if the head commit contains [s k i p c i] (w/o the extra spaces used to escape here), no jobs are executed.
In addition are all commits which modify non-code files (e.g. README) only excluded.
Furthermore we should not run cppcheck and scan-build if only profiles are changed and sort.py need only to be execute if profiles are changed.
* Create sort.yml
* Update build.yml
profile-sort is now in sort.yml
* Update sort.yml
fix syntax
* Update codeql-analysis.yml
paths-ignore:
- CONTRIBUTING.md
- README
- README.md
- RELNOTES
- SECURITY.md
- 'etc/**'
* Create build-extra.yml
* Update build.yml
* Update build-extra.yml
Diffstat (limited to '.github/workflows')
-rw-r--r-- | .github/workflows/build-extra.yml | 52 | ||||
-rw-r--r-- | .github/workflows/build.yml | 45 | ||||
-rw-r--r-- | .github/workflows/codeql-analysis.yml | 14 | ||||
-rw-r--r-- | .github/workflows/sort.yml | 21 |
4 files changed, 100 insertions, 32 deletions
diff --git a/.github/workflows/build-extra.yml b/.github/workflows/build-extra.yml new file mode 100644 index 000000000..1468ef898 --- /dev/null +++ b/.github/workflows/build-extra.yml | |||
@@ -0,0 +1,52 @@ | |||
1 | name: Build-extra CI | ||
2 | |||
3 | on: | ||
4 | push: | ||
5 | branches: [ master ] | ||
6 | paths-ignore: | ||
7 | - CONTRIBUTING.md | ||
8 | - README | ||
9 | - README.md | ||
10 | - RELNOTES | ||
11 | - SECURITY.md | ||
12 | - 'etc/**' | ||
13 | pull_request: | ||
14 | branches: [ master ] | ||
15 | paths-ignore: | ||
16 | - CONTRIBUTING.md | ||
17 | - README | ||
18 | - README.md | ||
19 | - RELNOTES | ||
20 | - SECURITY.md | ||
21 | - 'etc/**' | ||
22 | |||
23 | jobs: | ||
24 | build-clang: | ||
25 | if: ${{ ! contains(github.event.commits[0].message, '[skip ci]') }} | ||
26 | runs-on: ubuntu-20.04 | ||
27 | steps: | ||
28 | - uses: actions/checkout@v2 | ||
29 | - name: configure | ||
30 | run: CC=clang-10 ./configure --enable-fatal-warnings | ||
31 | - name: make | ||
32 | run: make | ||
33 | scan-build: | ||
34 | if: ${{ ! contains(github.event.commits[0].message, '[skip ci]') }} | ||
35 | runs-on: ubuntu-20.04 | ||
36 | steps: | ||
37 | - uses: actions/checkout@v2 | ||
38 | - name: install clang-tools-10 | ||
39 | run: sudo apt-get install clang-tools-10 | ||
40 | - name: configure | ||
41 | run: CC=clang-10 ./configure --enable-fatal-warnings | ||
42 | - name: scan-build | ||
43 | run: NO_EXTRA_CFLAGS="yes" scan-build-10 --status-bugs make | ||
44 | cppcheck: | ||
45 | if: ${{ ! contains(github.event.commits[0].message, '[skip ci]') }} | ||
46 | runs-on: ubuntu-20.04 | ||
47 | steps: | ||
48 | - uses: actions/checkout@v2 | ||
49 | - name: install cppcheck | ||
50 | run: sudo apt-get install cppcheck | ||
51 | - name: cppcheck | ||
52 | run: cppcheck -q --force --error-exitcode=1 --enable=warning,performance . | ||
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 71cb7f0b4..99b8a3be5 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml | |||
@@ -3,11 +3,24 @@ name: Build CI | |||
3 | on: | 3 | on: |
4 | push: | 4 | push: |
5 | branches: [ master ] | 5 | branches: [ master ] |
6 | paths-ignore: | ||
7 | - CONTRIBUTING.md | ||
8 | - README | ||
9 | - README.md | ||
10 | - RELNOTES | ||
11 | - SECURITY.md | ||
6 | pull_request: | 12 | pull_request: |
7 | branches: [ master ] | 13 | branches: [ master ] |
14 | paths-ignore: | ||
15 | - CONTRIBUTING.md | ||
16 | - README | ||
17 | - README.md | ||
18 | - RELNOTES | ||
19 | - SECURITY.md | ||
8 | 20 | ||
9 | jobs: | 21 | jobs: |
10 | build_and_test: | 22 | build_and_test: |
23 | if: ${{ ! contains(github.event.commits[0].message, '[skip ci]') }} | ||
11 | runs-on: ubuntu-20.04 | 24 | runs-on: ubuntu-20.04 |
12 | steps: | 25 | steps: |
13 | - uses: actions/checkout@v2 | 26 | - uses: actions/checkout@v2 |
@@ -21,35 +34,3 @@ jobs: | |||
21 | run: sudo make install | 34 | run: sudo make install |
22 | - name: run tests | 35 | - name: run tests |
23 | run: SHELL=/bin/bash make test-github | 36 | run: SHELL=/bin/bash make test-github |
24 | build-clang: | ||
25 | runs-on: ubuntu-20.04 | ||
26 | steps: | ||
27 | - uses: actions/checkout@v2 | ||
28 | - name: configure | ||
29 | run: CC=clang-10 ./configure --enable-fatal-warnings | ||
30 | - name: make | ||
31 | run: make | ||
32 | scan-build: | ||
33 | runs-on: ubuntu-20.04 | ||
34 | steps: | ||
35 | - uses: actions/checkout@v2 | ||
36 | - name: install clang-tools-10 | ||
37 | run: sudo apt-get install clang-tools-10 | ||
38 | - name: configure | ||
39 | run: CC=clang-10 ./configure --enable-fatal-warnings | ||
40 | - name: scan-build | ||
41 | run: NO_EXTRA_CFLAGS="yes" scan-build-10 --status-bugs make | ||
42 | cppcheck: | ||
43 | runs-on: ubuntu-20.04 | ||
44 | steps: | ||
45 | - uses: actions/checkout@v2 | ||
46 | - name: install cppcheck | ||
47 | run: sudo apt-get install cppcheck | ||
48 | - name: cppcheck | ||
49 | run: cppcheck -q --force --error-exitcode=1 --enable=warning,performance . | ||
50 | profile-sort: | ||
51 | runs-on: ubuntu-20.04 | ||
52 | steps: | ||
53 | - uses: actions/checkout@v2 | ||
54 | - name: check profiles | ||
55 | run: ./contrib/sort.py etc/*/{*.inc,*.net,*.profile} | ||
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index a37bbb5c7..301c7fad2 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml | |||
@@ -8,9 +8,23 @@ name: "CodeQL" | |||
8 | on: | 8 | on: |
9 | push: | 9 | push: |
10 | branches: [master] | 10 | branches: [master] |
11 | paths-ignore: | ||
12 | - CONTRIBUTING.md | ||
13 | - README | ||
14 | - README.md | ||
15 | - RELNOTES | ||
16 | - SECURITY.md | ||
17 | - 'etc/**' | ||
11 | pull_request: | 18 | pull_request: |
12 | # The branches below must be a subset of the branches above | 19 | # The branches below must be a subset of the branches above |
13 | branches: [master] | 20 | branches: [master] |
21 | paths-ignore: | ||
22 | - CONTRIBUTING.md | ||
23 | - README | ||
24 | - README.md | ||
25 | - RELNOTES | ||
26 | - SECURITY.md | ||
27 | - 'etc/**' | ||
14 | schedule: | 28 | schedule: |
15 | - cron: '0 7 * * 2' | 29 | - cron: '0 7 * * 2' |
16 | 30 | ||
diff --git a/.github/workflows/sort.yml b/.github/workflows/sort.yml new file mode 100644 index 000000000..55ac065b6 --- /dev/null +++ b/.github/workflows/sort.yml | |||
@@ -0,0 +1,21 @@ | |||
1 | name: sort.py | ||
2 | |||
3 | on: | ||
4 | push: | ||
5 | branches: [ master ] | ||
6 | paths: | ||
7 | - 'etc/**' | ||
8 | pull_request: | ||
9 | branches: [ master ] | ||
10 | paths: | ||
11 | - 'etc/**' | ||
12 | |||
13 | jobs: | ||
14 | profile-sort: | ||
15 | if: ${{ ! contains(github.event.commits[0].message, '[skip ci]') }} | ||
16 | runs-on: ubuntu-20.04 | ||
17 | steps: | ||
18 | - uses: actions/checkout@v2 | ||
19 | - name: check profiles | ||
20 | run: ./contrib/sort.py etc/*/{*.inc,*.profile} | ||
21 | |||