Treat Rust warnings as errors in code quality workflow

swisstopo · Nov 20, 2024 · 40f5ae5 · 40f5ae5
1 parent bc3e9b0
commit 40f5ae5
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/code-quality.yml b/.github/workflows/code-quality.yml
@@ -253,6 +253,9 @@ jobs:
           cd api
           cargo fmt --check
       - name: Run clippy
+        env:
+          # Treat warnings as errors
+          RUSTFLAGS: "-D warnings"
         run: |
           cd api
           cargo clippy --frozen --quiet

diff --git a/api/src/handlers.rs b/api/src/handlers.rs
@@ -87,7 +87,7 @@ pub struct Member {
 #[allow(non_snake_case)]
 #[serde(rename_all = "camelCase")]
 pub struct Geometry {
-    #[serde(rename="type")]
+    #[serde(rename = "type")]
     typ: String,
     positions: Vec<Cartesian3>,
     id: Option<String>,

diff --git a/api/src/lib.rs b/api/src/lib.rs
@@ -9,7 +9,7 @@ use axum::{
 use clap::Parser;
 use hyper::header::{ACCEPT, AUTHORIZATION, CONTENT_TYPE};
 use sqlx::PgPool;
-use tower::{ServiceBuilder};
+use tower::ServiceBuilder;
 use tower_http::{cors::CorsLayer, trace::TraceLayer};
 
 pub use config::Config;