feat: Trailing comma error message (#655)

* feat: adds macro check for trailing comma * feat: adds macro check for trailing comma * Update src/query.rs * feat: adds macro check for trailing comma --------- Co-authored-by: Predrag Gruevski <[email protected]>
obi1kenobi · Feb 11, 2024 · a6d0a00 · a6d0a00
1 parent 53dbd29
commit a6d0a00
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.gitignore b/.gitignore
@@ -13,6 +13,7 @@ localdata/
 
 # Editor-local configuration
 .vscode/
+.idea
 
 # Unnecessary test data
 test_crates/**/Cargo.lock
diff --git a/src/query.rs b/src/query.rs
@@ -444,7 +444,7 @@ mod tests {
 }
 
 macro_rules! add_lints {
-    ($($name:ident,)*) => {
+    ($($name:ident,)+) => {
         #[cfg(test)]
         mod tests_lints {
             $(
@@ -462,6 +462,9 @@ macro_rules! add_lints {
                 )*
             ]
         }
+    };
+    ($($name:ident),*) => {
+        compile_error!("Please add a trailing comma after each lint identifier. This ensures our scripts like 'make_new_lint.sh' can safely edit invocations of this macro as needed.");
     }
 }