fix(unknown_lints): Make emitted_source come before other groups

Author: Muscraft

src/cargo/util/lints.rs

@@ -535,6 +535,12 @@ fn output_unknown_lints(
         let help =
             matching.map(|(name, kind)| format!("there is a {kind} with a similar name: `{name}`"));
 
+        let mut footers = Vec::new();
+        if emitted_source.is_none() {
+            emitted_source = Some(UNKNOWN_LINTS.emitted_source(lint_level, reason));
+            footers.push(Level::Note.title(emitted_source.as_ref().unwrap()));
+        }
+
         let mut message = if let Some(span) =
             get_span(manifest.document(), &["lints", "cargo", lint_name], false)
         {
@@ -569,25 +575,21 @@ fn output_unknown_lints(
             } else {
                 Level::Note.title(&second_title)
             };
+            footers.push(inherited_note);
 
-            level
-                .title(&title)
-                .snippet(
-                    Snippet::source(ws_contents)
-                        .origin(&ws_path)
-                        .annotation(Level::Error.span(lint_span))
-                        .fold(true),
-                )
-                .footer(inherited_note)
+            level.title(&title).snippet(
+                Snippet::source(ws_contents)
+                    .origin(&ws_path)
+                    .annotation(Level::Error.span(lint_span))
+                    .fold(true),
+            )
         };
 
-        if emitted_source.is_none() {
-            emitted_source = Some(UNKNOWN_LINTS.emitted_source(lint_level, reason));
-            message = message.footer(Level::Note.title(emitted_source.as_ref().unwrap()));
-        }
-
         if let Some(help) = help.as_ref() {
-            message = message.footer(Level::Help.title(help));
+            footers.push(Level::Help.title(help));
+        }
+        for footer in footers {
+            message = message.footer(footer);
         }
 
         gctx.shell().print_message(message)?;

tests/testsuite/lints/unknown_lints.rs

@@ -76,13 +76,13 @@ workspace = true
 6 | this-lint-does-not-exist = "warn"
   | ^^^^^^^^^^^^^^^^^^^^^^^^
   |
+  = [NOTE] `cargo::unknown_lints` is set to `warn` by default
 [NOTE] `cargo::this-lint-does-not-exist` was inherited
  --> foo/Cargo.toml:9:1
   |
 9 | workspace = true
   | ----------------
   |
-  = [NOTE] `cargo::unknown_lints` is set to `warn` by default
 [CHECKING] foo v0.0.1 ([ROOT]/foo/foo)
 [FINISHED] `dev` profile [unoptimized + debuginfo] target(s) in [ELAPSED]s