From be66cc88acebe87e175659df1afc2e4586ed8958 Mon Sep 17 00:00:00 2001
From: Jay Kwak <82421531+jkwak-work@users.noreply.github.com>
Date: Thu, 18 Jul 2024 14:31:27 -0700
Subject: Enable warnings-as-error for CI (#4659)

Enable warnings-as-error for CI

Closes #4664.

Uses glslang commit that fixed a compiler warning
Fixes a recent warning from external/CMakelist.txt
A new macro, `SLANG_MAYBE_UNUSED` is added for a future need.
---
 .github/workflows/ci.yml                        | 10 +++++-----
 .github/workflows/compile-regression-test.yml   |  2 +-
 .github/workflows/falcor-compiler-perf-test.yml |  2 +-
 .github/workflows/falcor-test.yml               |  2 +-
 .github/workflows/vk-gl-cts-nightly.yml         |  2 +-
 5 files changed, 9 insertions(+), 9 deletions(-)

(limited to '.github')

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index cdf315d8e..a45d7647c 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -32,9 +32,11 @@ jobs:
           - { os: linux, runs-on: ubuntu-20.04 }
           - { os: macos, runs-on: macos-latest }
           - { os: windows, runs-on: windows-latest }
-          # When to have warnings
-          - warnings-as-errors: false
-          - { compiler: cl, warnings-as-errors true }
+          # Warnings are treated as errors by default.
+          # But we may want to disable it temporarily.
+          - { os: linux, warnings-as-errors: true }
+          - { os: macos, warnings-as-errors: true }
+          - { os: windows, warnings-as-errors: true }
           # Set a test category depending on the config, smoke by default,
           # quick or full conditionally otherwise
           - test-category: smoke
@@ -47,7 +49,6 @@ jobs:
             config: release
             compiler: gcc
             platform: aarch64
-            warnings-as-errors: false
             test-category: smoke
             full-gpu-tests: false
             runs-on: [self-hosted, Linux, ARM64]
@@ -56,7 +57,6 @@ jobs:
             config: release
             compiler: cl
             platform: x86_64
-            warnings-as-errors: false
             test-category: full
             full-gpu-tests: true
             runs-on: [Windows, self-hosted]
diff --git a/.github/workflows/compile-regression-test.yml b/.github/workflows/compile-regression-test.yml
index b6b484b9a..bd1011f86 100644
--- a/.github/workflows/compile-regression-test.yml
+++ b/.github/workflows/compile-regression-test.yml
@@ -21,7 +21,7 @@ jobs:
         platform: [x86_64]
         include:
           # Self-hosted falcor tests
-          - warnings-as-errors: false
+          - warnings-as-errors: true
             test-category: full
             full-gpu-tests: false
             runs-on: [Windows, self-hosted, regression-test]
diff --git a/.github/workflows/falcor-compiler-perf-test.yml b/.github/workflows/falcor-compiler-perf-test.yml
index 449944181..2fe168ade 100644
--- a/.github/workflows/falcor-compiler-perf-test.yml
+++ b/.github/workflows/falcor-compiler-perf-test.yml
@@ -23,7 +23,7 @@ jobs:
         platform: [x86_64]
         include:
           # Self-hosted falcor tests
-          - warnings-as-errors: false
+          - warnings-as-errors: true
             test-category: full
             full-gpu-tests: false
             runs-on: [Windows, self-hosted, perf]
diff --git a/.github/workflows/falcor-test.yml b/.github/workflows/falcor-test.yml
index 9dccb6af2..04c734bd4 100644
--- a/.github/workflows/falcor-test.yml
+++ b/.github/workflows/falcor-test.yml
@@ -21,7 +21,7 @@ jobs:
         platform: [x86_64]
         include:
           # Self-hosted falcor tests
-          - warnings-as-errors: false
+          - warnings-as-errors: true
             test-category: full
             full-gpu-tests: false
             runs-on: [Windows, self-hosted, falcor]
diff --git a/.github/workflows/vk-gl-cts-nightly.yml b/.github/workflows/vk-gl-cts-nightly.yml
index 374ea1b43..a835d20fc 100644
--- a/.github/workflows/vk-gl-cts-nightly.yml
+++ b/.github/workflows/vk-gl-cts-nightly.yml
@@ -18,7 +18,7 @@ jobs:
             compiler: cl
             platform: x86_64
             config: release
-            warnings-as-errors: false
+            warnings-as-errors: true
             test-category: full
             full-gpu-tests: false
             runs-on: [Windows, self-hosted]
-- 
cgit v1.2.3