Skip to content

Commit

Permalink
CI: check for c89 declaration
Browse files Browse the repository at this point in the history
to prevent this to be caught late, as happened with #85
  • Loading branch information
GitMensch committed Jun 9, 2023
1 parent cca51ff commit 9ee4b57
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ on:

jobs:
build:
name: Build, test and provide nightly
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -139,6 +140,7 @@ jobs:
coverage:
name: Coverage and Warnings
runs-on: ubuntu-latest

steps:
Expand All @@ -159,10 +161,11 @@ jobs:
export TERM="vt100"
echo "TERM=$TERM" >> $GITHUB_ENV
# note: w add additional C compiler syntax checks here to not need _another_ CI run
- name: configure
run: |
cd _build
../configure --enable-code-coverage
../configure --enable-code-coverage CPPFLAGS="-Werror=declaration-after-statement"
- uses: actions/upload-artifact@v3
if: failure()
Expand Down

0 comments on commit 9ee4b57

Please sign in to comment.