fix(format): return exit code 0 when formatting files

Format command should always return success exit code (0) when it
completes successfully, regardless of whether files needed formatting.
Only show "no issues found" message when no formatting was required.

Fixes regression introduced in 0d1f602 where format operations would
return exit code 1 when files needed formatting.

Author: djmccormick

package.json

@@ -16,6 +16,7 @@
     "Chase Holdren (https://github.com/chaseholdren)",
     "Chase Tamnoon (https://github.com/chase-tamnoon)",
     "Daniel Silva (https://github.com/dsilvasc)",
+    "Dustin McCormick (https://github.com/djmccormick)",
     "Elchin Valiyev (https://github.com/evaliyev)",
     "Gabriel Pereira Woitechen (https://github.com/wtchnm)",
     "Jamie Haywood (https://github.com/jamiehaywood)",

src/effects/format.rs

@@ -57,10 +57,8 @@ fn fix_formatting(ctx: Context) -> i32 {
       package.borrow().write_to_disk(&ctx.config);
     });
   }
-  if was_invalid {
-    1
-  } else {
+  if !was_invalid {
     ui::util::print_no_issues_found();
-    0
   }
+  0
 }