Omit values when they have the same name as the key

Author: neilvcarvalho

exe/importmap-update

@@ -67,7 +67,7 @@ end
 
 rails_root = ENV.fetch("RAILS_ROOT", ".")
 runner = Importmap::Update::Commands::ShellRunner.new(cwd: rails_root)
-gh = Importmap::Update::GitHubClient.new(repo: repo, token: token)
+gh = Importmap::Update::GitHubClient.new(repo:, token:)
 git = Importmap::Update::GitClient.new(
   repo: Git.open(rails_root),
   author_name: ENV.fetch("IMPORTMAP_AUTHOR_NAME", "github-actions[bot]"),
@@ -81,10 +81,10 @@ plan = Importmap::Update::Planner.new(
 ).call
 
 existing_prs = gh.list_open_prs(branch_prefix: config.branch_prefix)
-reconciled = Importmap::Update::Reconciler.new(plan: plan, existing_prs: existing_prs).call
+reconciled = Importmap::Update::Reconciler.new(plan:, existing_prs:).call
 
 executor = Importmap::Update::Executor.new(
-  gh: gh, git: git, runner: runner,
+  gh:, git:, runner:,
   base_branch: ENV.fetch("IMPORTMAP_BASE_BRANCH", "main"),
   commit_message_prefix: config.commit_message.prefix,
   labels: config.labels,

lib/commands.rb

@@ -47,7 +47,7 @@ def run(*argv)
           opts[:chdir] = @cwd if @cwd
           Bundler.with_unbundled_env do
             stdout, stderr, status = Open3.capture3(*argv, opts)
-            Result.new(stdout: stdout, stderr: stderr, exit_code: status.exitstatus)
+            Result.new(stdout:, stderr:, exit_code: status.exitstatus)
           end
         end
 

lib/config.rb

@@ -62,13 +62,13 @@ def self.default
 
       def to_h
         {
-          version: version,
+          version:,
           grouping: grouping.transform_values { |b| {strategy: b.strategy} },
-          open_pull_requests_limit: open_pull_requests_limit,
-          labels: labels,
-          reviewers: reviewers,
-          commit_message: {prefix: commit_message.prefix},
-          branch_prefix: branch_prefix
+          open_pull_requests_limit:,
+          labels:,
+          reviewers:,
+          branch_prefix:,
+          commit_message: {prefix: commit_message.prefix}
         }
       end
 

lib/git_client.rb

@@ -47,7 +47,7 @@ def commit_changes(message:)
       end
 
       def push(branch:, force: false)
-        @repo.push("origin", branch, force: force)
+        @repo.push("origin", branch, force:)
         nil
       end
     end

lib/github_client.rb

@@ -62,7 +62,7 @@ def ensure_labels(labels)
 
       # Edits an existing PR's title and body.
       def update_pr(number:, title:, body:)
-        @client.update_pull_request(@repo, number, title: title, body: body)
+        @client.update_pull_request(@repo, number, title:, body:)
         nil
       end
 

lib/parsers/audit_parser.rb

@@ -62,11 +62,11 @@ def split_row(line)
         # If a description contained a `|`, cells.size will be >4. Rejoin the
         # tail into the advisory column so we don't lose information.
         def build_row(cells)
-          name, severity, vuln_versions, *advisory_parts = cells
+          name, severity, vulnerable_versions, *advisory_parts = cells
           Vulnerability.new(
-            name: name,
-            severity: severity,
-            vulnerable_versions: vuln_versions,
+            name:,
+            severity:,
+            vulnerable_versions:,
             advisory: advisory_parts.join(" | ")
           )
         end

lib/parsers/outdated_parser.rb

@@ -71,9 +71,9 @@ def split_row(line)
         def build_row(cells)
           name, current, latest_or_error = cells[0], cells[1], cells[2]
           if VERSION_SHAPE_RE.match?(latest_or_error)
-            OutdatedPackage.new(name: name, current: current, latest: latest_or_error, error: nil)
+            OutdatedPackage.new(name:, current:, latest: latest_or_error, error: nil)
           else
-            OutdatedPackage.new(name: name, current: current, latest: nil, error: latest_or_error)
+            OutdatedPackage.new(name:, current:, latest: nil, error: latest_or_error)
           end
         end
       end

test/executor_test.rb

@@ -30,19 +30,19 @@ def ensure_labels(labels)
     end
 
     def create_pr(branch:, base:, title:, body:, labels: [])
-      @created << {branch: branch, base: base, title: title, body: body, labels: labels}
+      @created << {branch:, base:, title:, body:, labels:}
       n = @next_pr_number
       @next_pr_number += 1
       n
     end
 
     def update_pr(number:, title:, body:)
-      @updated << {number: number, title: title, body: body}
+      @updated << {number:, title:, body:}
       nil
     end
 
     def close_pr(number:, comment: nil)
-      @closed << {number: number, comment: comment}
+      @closed << {number:, comment:}
       nil
     end
   end
@@ -79,12 +79,12 @@ def push(branch:, force: false)
 
   def bump(name, from, to, kind: :patch, severity: nil)
     advisory = severity ? {severity: severity} : nil
-    Planner::PackageBump.new(name: name, from: from, to: to, semver_kind: kind, advisory: advisory)
+    Planner::PackageBump.new(name:, from:, to:, semver_kind: kind, advisory:)
   end
 
   def spec(branch:, packages:, kind: :patch, title: "spec title")
     Planner::PRSpec.new(
-      kind: kind, packages: packages, branch: branch, title: title,
+      kind:, packages:, branch:, title:,
       metadata: {
         tool: "importmap-update", kind: kind,
         packages: packages.map { |p| {name: p.name, from: p.from, to: p.to, semver_kind: p.semver_kind} }
@@ -93,7 +93,7 @@ def spec(branch:, packages:, kind: :patch, title: "spec title")
   end
 
   def existing_pr(number:, branch:)
-    Reconciler::ExistingPR.new(number: number, branch: branch, body: "", title: "old")
+    Reconciler::ExistingPR.new(number:, branch:, body: "", title: "old")
   end
 
   def setup