diff options
author | Yusuke Tanaka <yusuktan@maguro.dev> | 2023-03-02 17:50:17 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-03-02 21:50:17 +0000 |
commit | 88c9a999f78916700b35b6a893a5b779a4f86db0 (patch) | |
tree | 0bb71bfa024f9262ee71c98333cc1a6eeace67e9 /cli/tools/lint.rs | |
parent | 55833cf799979e63c6b027fbbf018272308caf5c (diff) |
chore(cli): update deno_lint to 0.41.0 (#17997)
<!--
Before submitting a PR, please read http://deno.land/manual/contributing
1. Give the PR a descriptive title.
Examples of good title:
- fix(std/http): Fix race condition in server
- docs(console): Update docstrings
- feat(doc): Handle nested reexports
Examples of bad title:
- fix #7123
- update docs
- fix bugs
2. Ensure there is a related issue and it is referenced in the PR text.
3. Ensure there are tests that cover the changes.
4. Ensure `cargo test` passes.
5. Ensure `./tools/format.js` passes without changing files.
6. Ensure `./tools/lint.js` passes.
7. Open as a draft PR if your work is still in progress. The CI won't
run
all steps, but you can add '[ci]' to a commit message to force it to.
8. If you would like to run the benchmarks on the CI, add the 'ci-bench'
label.
-->
This commit updates deno_lint crate to 0.41.0. The new version contains
a braking change that requries a minor code fix here, which is also
addressed in this commit.
Diffstat (limited to 'cli/tools/lint.rs')
-rw-r--r-- | cli/tools/lint.rs | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/cli/tools/lint.rs b/cli/tools/lint.rs index 0f81ec89d..4eebf9e16 100644 --- a/cli/tools/lint.rs +++ b/cli/tools/lint.rs @@ -234,7 +234,7 @@ pub fn print_rules_list(json: bool) { pub fn create_linter( media_type: MediaType, - rules: Vec<Arc<dyn LintRule>>, + rules: Vec<&'static dyn LintRule>, ) -> Linter { LinterBuilder::default() .ignore_file_directive("deno-lint-ignore-file") @@ -247,7 +247,7 @@ pub fn create_linter( fn lint_file( file_path: &PathBuf, source_code: String, - lint_rules: Vec<Arc<dyn LintRule>>, + lint_rules: Vec<&'static dyn LintRule>, ) -> Result<(Vec<LintDiagnostic>, String), AnyError> { let file_name = file_path.to_string_lossy().to_string(); let media_type = MediaType::from(file_path); @@ -263,7 +263,7 @@ fn lint_file( /// Treats input as TypeScript. /// Compatible with `--json` flag. fn lint_stdin( - lint_rules: Vec<Arc<dyn LintRule>>, + lint_rules: Vec<&'static dyn LintRule>, ) -> Result<(Vec<LintDiagnostic>, String), AnyError> { let mut source_code = String::new(); if stdin().read_to_string(&mut source_code).is_err() { @@ -530,7 +530,9 @@ fn sort_diagnostics(diagnostics: &mut [LintDiagnostic]) { }); } -pub fn get_configured_rules(rules: LintRulesConfig) -> Vec<Arc<dyn LintRule>> { +pub fn get_configured_rules( + rules: LintRulesConfig, +) -> Vec<&'static dyn LintRule> { if rules.tags.is_none() && rules.include.is_none() && rules.exclude.is_none() { rules::get_recommended_rules() |