Skip to content

Commit ba8b9bf

Browse files
authored
ci: remove clang report page
The warnings found by clang report is already preventatively checked in the neovim repository: https://github.com/neovim/neovim/blob/7fa24948a936a95519f0c8c496402488b6508c14/.github/workflows/test.yml#L82-L95. This means that new warnings will not be passed in and the need for the report page is greatly reduced. Furthermore, some of the warnings such as core.NullDereference seem susceptible to false positives which are hard to identify and should be ignored for the time being. `clang-analyzer` gives better tools compared to `scan-build` to selectively ignore these warnings. Lastly, this will reduce the `reports` workflow from around 9 minutes to 1 minute.
1 parent 8c3865b commit ba8b9bf

File tree

5 files changed

+0
-117
lines changed

5 files changed

+0
-117
lines changed

.github/workflows/reports.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -50,15 +50,6 @@ jobs:
5050
sudo apt-get update
5151
sudo apt-get install -y build-essential cmake gcc-multilib gettext ninja-build clang-tools
5252
53-
- name: Update clang report
54-
env:
55-
CC: clang
56-
run: |
57-
${GITHUB_WORKSPACE}/ci/clang-report.sh
58-
cd "$DOC_DIR"
59-
git add --all .
60-
git commit -m 'clang report: Automatic update'
61-
6253
- name: Update vimpatch report
6354
run: |
6455
python -m pip install --user requests

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ This repo contains automation scripts and CI configuration to run the scripts.
1010
- [doc/ landing page](https://neovim.io/doc)
1111
- [user/ docs](https://neovim.io/doc/user)
1212
- Build reports:
13-
- [Clang report](https://neovim.io/doc/reports/clang)
1413
- [Vimpatch report](https://neovim.io/doc/reports/vimpatch)
1514

1615
## Run Locally

ci/clang-report.sh

Lines changed: 0 additions & 96 deletions
This file was deleted.

templates/clang-report/head.html

Lines changed: 0 additions & 8 deletions
This file was deleted.

templates/clang-report/no-warnings.html

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)