Merge pull request #546 from napi-rs/lint-exit-code

fix(deno-lint): has_error value

Author: Brooooooklyn

packages/deno-lint/Cargo.toml

@@ -10,8 +10,8 @@ crate-type = ["cdylib"]
 [dependencies]
 annotate-snippets = {version = "0.9", features = ["color"]}
 anyhow = "1"
-deno_ast = "0.4.1"
-deno_lint = "0.18.1"
+deno_ast = "0.5.0"
+deno_lint = "0.19.0"
 env_logger = "0.9"
 global_alloc = {path = "../../crates/alloc"}
 globwalk = "0.8"

packages/deno-lint/cli.ts

@@ -12,7 +12,7 @@ class LintCommand extends Command {
 
   private readonly checkOnly = Option.Boolean('--check-only', { required: false })
 
-  execute() {
+  async execute() {
     const hasError = denolint(this.cwd ?? __dirname, this.configPath ?? '.denolint.json')
     return Promise.resolve(hasError && !this.checkOnly ? 1 : 0)
   }

packages/deno-lint/src/lib.rs

@@ -182,7 +182,7 @@ fn lint_command(ctx: CallContext) -> Result<JsBoolean> {
           status: Status::GenericFailure,
           reason: format!("Lint failed: {}, at: {:?}", e, &p),
         })?;
-      has_error = !file_diagnostics.is_empty();
+      has_error = has_error || !file_diagnostics.is_empty();
       diagnostics::display_diagnostics(&file_diagnostics, s.source(), true);
     }
   }

rust-toolchain

@@ -1 +1 @@
-nightly-2021-10-21
+nightly-2021-11-01