Reorder migration results (#2835)

Show message about partial or skipped snippets first.

Author: AndreasArvidsson

packages/cursorless-vscode/src/migrateSnippets.ts

@@ -181,28 +181,33 @@ async function openResultDocument(
     `From: ${sourceDirectory}`,
     `To:   ${targetDirectory}`,
     "",
-    `## Migrated ${migratedKeys.length} snippet files:`,
-    ...migratedKeys.map((key) => `- ${key} -> ${result.migrated[key]}`),
-    "",
   ];
 
+  if (result.skipped.length > 0) {
+    content.push(
+      `## Skipped ${result.skipped.length} snippet files:`,
+      ...result.skipped.map((key) => `- ${key}`),
+      skipMessage,
+      "",
+    );
+  }
+
   if (migratedPartiallyKeys.length > 0) {
     content.push(
       `## Migrated ${migratedPartiallyKeys.length} snippet files partially:`,
       ...migratedPartiallyKeys.map(
         (key) => `- ${key} -> ${result.migratedPartially[key]}`,
       ),
       skipMessage,
+      "",
     );
   }
 
-  if (result.skipped.length > 0) {
-    content.push(
-      `## Skipped ${result.skipped.length} snippet files:`,
-      ...result.skipped.map((key) => `- ${key}`),
-      skipMessage,
-    );
-  }
+  content.push(
+    `## Migrated ${migratedKeys.length} snippet files:`,
+    ...migratedKeys.map((key) => `- ${key} -> ${result.migrated[key]}`),
+    "",
+  );
 
   const textDocument = await vscode.workspace.openTextDocument({
     content: content.join("\n"),