Skip to content

Commit

Permalink
fix(core): linters missing descriptions
Browse files Browse the repository at this point in the history
  • Loading branch information
elijah-potter committed Jan 22, 2025
1 parent 374920a commit f7a9858
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion harper-core/src/linting/lets_confusion/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ use super::merge_linters::merge_linters;
use let_us_redundancy::LetUsRedundancy;
use no_contraction_with_verb::NoContractionWithVerb;

merge_linters!(LetsConfusion => LetUsRedundancy, NoContractionWithVerb => "TODO");
merge_linters!(LetsConfusion => LetUsRedundancy, NoContractionWithVerb => "It's often hard to determine where the subject should go with the word `let`. This rule attempts to find common errors with redundancy and contractions that may lead to confusion for readers.");

#[cfg(test)]
mod tests {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,6 @@ impl PatternLinter for NoContractionWithVerb {
}

fn description(&self) -> &'static str {
"Make sure you include a subject."
"Make sure you include a subject when giving permission to it."
}
}

0 comments on commit f7a9858

Please sign in to comment.