Remove dump command from rewatch cli (#7710)

Author: cknitt

CHANGELOG.md

@@ -15,6 +15,7 @@
 #### :boom: Breaking Change
 
 - `Result.getOrThrow` now throws a JS error instead of a `Not_found` ReScript exception. https://github.com/rescript-lang/rescript/pull/7630
+- Remove `rescript dump` command. `bsc` can be used directly to dump the contents of a `.cmi` file instead if needed. https://github.com/rescript-lang/rescript/pull/7710
 
 #### :rocket: New Feature
 

rewatch/src/cli.rs

@@ -203,12 +203,6 @@ pub enum Command {
         #[arg(group = "format_input_mode", required_unless_present_any = ["format_input_mode"])]
         files: Vec<String>,
     },
-    /// Alias to `legacy dump`.
-    #[command(disable_help_flag = true)]
-    Dump {
-        #[arg(allow_hyphen_values = true, num_args = 0..)]
-        dump_args: Vec<OsString>,
-    },
     /// This prints the compiler arguments. It expects the path to a rescript file (.res or .resi).
     CompilerArgs {
         /// Path to a rescript file (.res or .resi)

rewatch/src/main.rs

@@ -97,11 +97,6 @@ fn main() -> Result<()> {
             check,
             files,
         } => format::format(stdin, all, check, files),
-        cli::Command::Dump { mut dump_args } => {
-            dump_args.insert(0, "dump".into());
-            let code = build::pass_through_legacy(dump_args);
-            std::process::exit(code);
-        }
     }
 }