Clippy test

.github/workflows/rust.yml

@@ -1,4 +1,4 @@
-name: Rust
+name: Rust quality test
 
 on:
   push:
@@ -8,6 +8,7 @@ on:
 
 env:
   CARGO_TERM_COLOR: always
+  RUSTFLAGS: "-Dwarnings" # Make sure CI fails on all warnings, including Clippy lints
 
 jobs:
   build:
@@ -16,6 +17,9 @@ jobs:
 
     steps:
     - uses: actions/checkout@v4
+    -name: Run Clippy
+      working-directory: ./quality
+      run: cargo clippy --all-targets --all-features
     - name: Build
       working-directory: ./quality
       run: cargo build --verbose

Dojo-101-DevSec/Rust-security.md

@@ -16,6 +16,14 @@ démo des fonctions de sécurité de RUST
 rustc.exe .\Rust-security.rs
 ```
 
+## Toolchains
+
+la version stable est recommandée
+
+```powershell
+rustup toolchain list
+```
+
 ## Créer un projet avec dépendances
 
 ```powershell
@@ -28,6 +36,16 @@ Les versions des dépendances sont alors spécifié dans le fichier `Cargo.toml`
 Pour mettre à jour des dépendances : `cargo-update`
 
 
+## Test & linter
+
+[Clippy](https://github.com/rust-lang/rust-clippy)
+
+Installé par défaut, si non `rustup component add clippy` (priviliger cette commande à `cargo` pour ne pas l'avoir en dépendance)
+
+```
+cargo clippy
+```
+
 ## Test via CI/CD
 
 ### github actions
@@ -162,5 +180,3 @@ fn concurrency() {
     println!("Result: {}", *counter.lock().unwrap());
 }
 ```
-
-### dépendances

quality/src/main.rs

@@ -29,7 +29,7 @@ fn main() {
     let mut files = vec![];
     let mut non_markdown_files = vec![];
 
-    for entry in WalkDir::new(&parent_dir).min_depth(2).max_depth(3) {
+    for entry in WalkDir::new(parent_dir).min_depth(2).max_depth(3) {
         let entry = entry.unwrap();
         let path = entry.path();
 
@@ -63,7 +63,7 @@ fn main() {
 // 2. Vérifie qu'il n'y a pas de dossier dans les sous dossiers
 
 
-    for entry in WalkDir::new(&parent_dir).min_depth(1).max_depth(1) {
+    for entry in WalkDir::new(parent_dir).min_depth(1).max_depth(1) {
         let entry = entry.unwrap();
         let path = entry.path();
         if path.is_dir() {