Skip to content

Commit

Permalink
Auto merge of #8061 - vallentin:fix-same-name-method-desc, r=flip1995
Browse files Browse the repository at this point in the history
Fixed same_name_method description

Noticed some odd phrasing, while checking out the new release.

changelog: none
  • Loading branch information
bors committed Dec 2, 2021
2 parents 392b0c5 + aec74d2 commit 78036a8
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions clippy_lints/src/same_name_method.rs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ use std::collections::{BTreeMap, BTreeSet};

declare_clippy_lint! {
/// ### What it does
/// It lints if a struct has two method with same time:
/// It lints if a struct has two methods with the same name:
/// one from a trait, another not from trait.
///
/// ### Why is this bad?
Expand Down Expand Up @@ -100,7 +100,7 @@ impl<'tcx> LateLintPass<'tcx> for SameNameMethod {
cx,
SAME_NAME_METHOD,
*impl_span,
"method's name is same to an existing method in a trait",
"method's name is the same as an existing method in a trait",
|diag| {
diag.span_note(
trait_method_span,
Expand Down Expand Up @@ -139,7 +139,7 @@ impl<'tcx> LateLintPass<'tcx> for SameNameMethod {
cx,
SAME_NAME_METHOD,
impl_span,
"method's name is same to an existing method in a trait",
"method's name is the same as an existing method in a trait",
|diag| {
// TODO should we `span_note` on every trait?
// iterate on trait_spans?
Expand Down

0 comments on commit 78036a8

Please sign in to comment.