feat(cli): support multiple output formats via --out

Parse comma-separated formats in --out and set Skunk::Config.formats.\n\nBreaking change: replaces previous --out=FILE behavior.\nDocs: updated help text and examples in README; added CHANGELOG entry.

Author: JuanVqz

CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## main [(unreleased)](https://github.com/fastruby/skunk/compare/v0.5.4...HEAD)
 
+* [FEATURE: `--out` now accepts multiple formats and sets `Skunk::Config.formats`](#)
+* [BREAKING: Replace `--out=FILE` file output with formats configuration (`--out=json,html`) ](#)
+
 * [REFACTOR: Move Console Report](https://github.com/fastruby/skunk/pull/128)
 * [BUGFIX: Set the right content type in the share HTTP request](https://github.com/fastruby/skunk/pull/129)
 * [REFACTOR: Centralize Skunk analysis into RubyCritic module](https://github.com/fastruby/skunk/pull/127)

README.md

@@ -65,7 +65,7 @@ Run `skunk -h` to check out the help options:
 ```
 Usage: skunk [options] [paths]
     -b, --branch BRANCH              Set branch to compare
-    -o, --out FILE                   Output report to file
+    -o, --out console,json,html      Output formats
     -v, --version                    Show gem's version
     -h, --help                       Show this message
 ```
@@ -161,8 +161,9 @@ This should give you an idea if you're moving in the direction of maintaining th
 Skunk provides a simple configuration class to control output formats programmatically. You can use `Skunk::Config` to set which formats should be generated when running Skunk.
 
 **Supported formats:**
-- `:json` - JSON report (default)
-- `:html` - HTML report with visual charts and tables
+- `:console` - Console report (default)
+- `:json`    - JSON report
+- `:html`    - HTML report with visual charts and tables
 
 ```ruby
 require 'skunk/config'
@@ -184,6 +185,13 @@ Skunk::Config.supported_format?(:json) # => true
 Skunk::Config.reset
 ```
 
+You can also configure formats via the CLI using `--out` with a comma-separated list:
+
+```
+skunk --out=json,html
+skunk -o console
+```
+
 ## Sharing your SkunkScore
 
 If you want to share the results of your Skunk report with the Ruby community, run:

lib/skunk/cli/options/argv.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "rubycritic/cli/options/argv"
+require "skunk/config"
 
 module Skunk
   module Cli
@@ -22,8 +23,8 @@ def parse
               self.mode = :compare_branches
             end
 
-            opts.on("-o", "--out FILE", "Output report to file") do |filename|
-              self.output_filename = filename
+            opts.on("-o", "--out console,json,html", Array, "Output formats") do |formats|
+              Skunk::Config.formats = formats.map(&:to_sym)
             end
 
             opts.on_tail("-v", "--version", "Show gem's version") do

test/lib/skunk/cli/options/argv_test.rb

@@ -3,24 +3,27 @@
 require "test_helper"
 
 require "skunk/cli/options/argv"
+require "skunk/config"
 
 describe Skunk::Cli::Options::Argv do
-  describe "#output_filename" do
-    context "passing --out=FILE options" do
-      let(:argv) { ["--out=file.txt"] }
+  describe "--out formats" do
+    context "passing --out=json,html" do
+      let(:argv) { ["--out=json,html"] }
 
-      it "parses passed filename" do
+      it "sets Config.formats accordingly" do
+        Skunk::Config.reset
         parser = Skunk::Cli::Options::Argv.new(argv)
         parser.parse
-        _(parser.output_filename).must_equal "file.txt"
+        _(Skunk::Config.formats).must_equal %i[json html]
       end
     end
 
     context "not passing the --out option" do
-      it "is nil" do
+      it "keeps default formats" do
+        Skunk::Config.reset
         parser = Skunk::Cli::Options::Argv.new([])
         parser.parse
-        _(parser.output_filename).must_be_nil
+        _(Skunk::Config.formats).must_equal [:console]
       end
     end
   end