Fix CI

Author: pmnoxx

clippy_lints/src/lib.register_nursery.rs

@@ -18,7 +18,6 @@ store.register_group(true, "clippy::nursery", Some("clippy_nursery"), vec![
     LintId::of(missing_const_for_fn::MISSING_CONST_FOR_FN),
     LintId::of(mutable_debug_assertion::DEBUG_ASSERT_WITH_MUT_CALL),
     LintId::of(mutex_atomic::MUTEX_INTEGER),
-    LintId::of(methods::CLONED_LAST),
     LintId::of(non_send_fields_in_send_ty::NON_SEND_FIELDS_IN_SEND_TY),
     LintId::of(nonstandard_macro_braces::NONSTANDARD_MACRO_BRACES),
     LintId::of(option_if_let_else::OPTION_IF_LET_ELSE),

clippy_lints/src/methods/cloned_last.rs

@@ -16,15 +16,15 @@ pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx hir::Expr<'_>, rec
     });
     if recv_impls_iterator {
         let msg = "called `cloned().last()` on an `Iterator`. It may be more efficient to call
-                   `.last().clone()` instead";
+                   `.last().cloned()` instead";
         let iter_snippet = snippet(cx, recv.span, "..");
         span_lint_and_sugg(
             cx,
             CLONED_LAST,
             expr.span,
             msg,
             "try this",
-            format!("{}.last().clone()", iter_snippet),
+            format!("{}.last().cloned()", iter_snippet),
             Applicability::MachineApplicable,
         );
     }

clippy_lints/src/methods/mod.rs

@@ -123,7 +123,7 @@ declare_clippy_lint! {
     /// Could be written as
     /// ```rust
     /// # let vec = vec!["string".to_string()];
-    /// vec.iter().last().clone();
+    /// vec.iter().last().cloned();
     /// ```
     #[clippy::version = "1.59.0"]
     pub CLONED_LAST,

tests/ui/cloned_last.fixed

@@ -1,8 +1,8 @@
 // run-rustfix
-#![warn(clippy::nursery)]
+#![warn(clippy::all)]
 
 fn main() {
     #[rustfmt::skip]
     let _: Option<String> = vec!["1".to_string(), "2".to_string(), "3".to_string()]
-        .iter().last().clone();
+        .iter().last().cloned();
 }

tests/ui/cloned_last.rs

@@ -1,5 +1,5 @@
 // run-rustfix
-#![warn(clippy::nursery)]
+#![warn(clippy::all)]
 
 fn main() {
     #[rustfmt::skip]

tests/ui/cloned_last.stderr

@@ -1,5 +1,5 @@
 error: called `cloned().last()` on an `Iterator`. It may be more efficient to call
-                          `.last().clone()` instead
+                          `.last().cloned()` instead
   --> $DIR/cloned_last.rs:6:29
    |
 LL |       let _: Option<String> = vec!["1".to_string(), "2".to_string(), "3".to_string()]
@@ -13,7 +13,7 @@ LL | |         .last();
 help: try this
    |
 LL ~     let _: Option<String> = vec!["1".to_string(), "2".to_string(), "3".to_string()]
-LL ~         .iter().last().clone();
+LL ~         .iter().last().cloned();
    |
 
 error: aborting due to previous error