fix: do not return failure code on warning

Fail only if there are errors, let warnings pass.
supabase-community · Nov 14, 2024 · 8ccf6d7 · 8ccf6d7
1 parent d4ca8ca
commit 8ccf6d7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -21,6 +21,7 @@ mod rules;
 mod utils;
 
 pub use crate::config::Config;
+pub use crate::errors::LintLevel;
 pub use crate::output::{rdf::RdfFormatter, simple::SimpleFormatter, LintOutput, OutputFormatter};
 pub use crate::utils::is_lintable;
 

diff --git a/src/main.rs b/src/main.rs
@@ -11,7 +11,7 @@ use glob::glob;
 use log::{debug, error};
 use simplelog::{ColorChoice, Config as LogConfig, LevelFilter, TermLogger, TerminalMode};
 use supa_mdx_lint::{
-    is_lintable, Config, LintOutput, LintTarget, Linter, LinterBuilder, OutputFormatter,
+    is_lintable, Config, LintLevel, LintOutput, LintTarget, Linter, LinterBuilder, OutputFormatter,
 };
 
 const DEFAULT_CONFIG_FILE: &str = "supa-mdx-lint.config.toml";
@@ -145,7 +145,10 @@ fn execute() -> Result<Result<()>> {
 
     stdout.flush()?;
 
-    if diagnostics.iter().any(|d| !d.errors().is_empty()) {
+    if diagnostics
+        .iter()
+        .any(|d| d.errors().iter().any(|e| e.level == LintLevel::Error))
+    {
         Ok(Err(anyhow::anyhow!("Linting errors found")))
     } else {
         Ok(Ok(()))