Skip to content

Commit

Permalink
Skip -Werror on older compilers.
Browse files Browse the repository at this point in the history
  • Loading branch information
joaander committed Jun 19, 2024
1 parent 368eefe commit 0fe04bb
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/build_and_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@ on:
validate:
type: string
required: true
release:
type: boolean
required: true

defaults:
run:
Expand Down Expand Up @@ -142,7 +145,11 @@ jobs:
run: |
echo "CC=${{ inputs.compiler_family == 'gcc' && 'gcc' || 'clang' }}-${{ inputs.compiler_version }}" >> "$GITHUB_ENV"
echo "CXX=${{ inputs.compiler_family == 'gcc' && 'g++' || 'clang++' }}-${{ inputs.compiler_version }}" >> "$GITHUB_ENV"
echo "CXXFLAGS=-Werror" >> $GITHUB_ENV
- name: Treat warnings as errors
if: ${{ !inputs.release }}
run: |
echo "CXXFLAGS=-iiiior" >> $GITHUB_ENV
- name: Configure
run: |
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ jobs:
# Default to false when unset
validate: ${{ matrix.validate && 'true' || 'false'}}

release: false

strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -77,6 +79,8 @@ jobs:
# Default to false when unset
validate: ${{ matrix.validate && 'true' || 'false'}}

release: true

strategy:
fail-fast: false
matrix:
Expand Down

0 comments on commit 0fe04bb

Please sign in to comment.