Fix --no-user-output not working anymore

Conversion of the output format argument was only happening for `Context` but
not for the direct access in `cargo-msrv.rs`.

src/bin/cargo-msrv.rs

@@ -71,7 +71,7 @@ fn setup_reporter(opts: CargoMsrvOpts) -> Result<ExitCode, SetupError> {
 
     tracing::info!("storyteller channel created");
 
-    let output_format = opts.shared_opts.user_output_opts.output_format;
+    let output_format = opts.shared_opts.user_output_opts.effective_output_format();
     let handler = WrappingHandler::from(output_format);
     let finalizer = listener.run_handler(Arc::new(handler));
     tracing::info!("storyteller started handler");

src/cli/shared_opts.rs

@@ -33,11 +33,21 @@ pub struct UserOutputOpts {
         value_name = "FORMAT",
         global = true
     )]
-    pub output_format: OutputFormat,
+    output_format: OutputFormat,
 
     /// Disable user output
     #[arg(long, global = true)]
-    pub no_user_output: bool,
+    no_user_output: bool,
+}
+
+impl UserOutputOpts {
+    pub fn effective_output_format(&self) -> OutputFormat {
+        if self.no_user_output {
+            OutputFormat::None
+        } else {
+            self.output_format
+        }
+    }
 }
 
 #[derive(Debug, Args)]

src/context.rs

@@ -305,14 +305,8 @@ pub struct UserOutputContext {
 
 impl From<UserOutputOpts> for UserOutputContext {
     fn from(opts: UserOutputOpts) -> Self {
-        if opts.no_user_output {
-            Self {
-                output_format: OutputFormat::None,
-            }
-        } else {
-            Self {
-                output_format: opts.output_format,
-            }
+        Self {
+            output_format: opts.effective_output_format(),
         }
     }
 }