fixing rubocop

Author: dutow

app/controllers/admin/mailing_lists_controller.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 class Admin::MailingListsController < Admin::BaseController
-  before_action :set_mailing_list, only: [:edit, :update]
+  before_action :set_mailing_list, only: [ :edit, :update ]
 
   def active_admin_section
     :mailing_lists
@@ -16,7 +16,7 @@ def index
       )
       .order(:display_name)
 
-    @topic_list_distribution = TopicMailingList.connection.select_rows(<<~SQL).to_h { |c, t| [c.to_i, t.to_i] }
+    @topic_list_distribution = TopicMailingList.connection.select_rows(<<~SQL).to_h { |c, t| [ c.to_i, t.to_i ] }
       SELECT lists_count, COUNT(*) AS topic_count
       FROM (SELECT COUNT(*) AS lists_count FROM topic_mailing_lists GROUP BY topic_id) counts
       GROUP BY lists_count ORDER BY lists_count

app/models/mailing_list.rb

@@ -8,7 +8,7 @@ class MailingList < ApplicationRecord
   validates :display_name, presence: true
 
   def all_emails
-    [email, *alternate_emails].compact.reject(&:blank?).map(&:downcase)
+    [ email, *alternate_emails ].compact.reject(&:blank?).map(&:downcase)
   end
 
   def self.email_lookup_index

db/migrate/20260324054041_create_mailing_list_tables.rb

@@ -15,14 +15,14 @@ def up
       t.references :mailing_list, null: false, foreign_key: true
       t.timestamps
     end
-    add_index :message_mailing_lists, [:message_id, :mailing_list_id], unique: true, name: "idx_message_mailing_lists_unique"
+    add_index :message_mailing_lists, [ :message_id, :mailing_list_id ], unique: true, name: "idx_message_mailing_lists_unique"
 
     create_table :topic_mailing_lists do |t|
       t.references :topic, null: false, foreign_key: true
       t.references :mailing_list, null: false, foreign_key: true
       t.timestamps
     end
-    add_index :topic_mailing_lists, [:topic_id, :mailing_list_id], unique: true, name: "idx_topic_mailing_lists_unique"
+    add_index :topic_mailing_lists, [ :topic_id, :mailing_list_id ], unique: true, name: "idx_topic_mailing_lists_unique"
 
     hackers = execute <<~SQL
       INSERT INTO mailing_lists (identifier, display_name, email, created_at, updated_at)

script/imap_idle.rb

@@ -48,7 +48,7 @@
 Signal.trap('INT')  { stop_w.write_nonblock("\0") rescue nil }
 
 Thread.new do
-  IO.select([stop_r])
+  IO.select([ stop_r ])
   stop_r.close
   logger.info('[imap_idle] Signal received, stopping...') if logger
   runner.stop!

spec/models/mailing_list_spec.rb

@@ -12,7 +12,7 @@
   describe ".email_lookup_index" do
     it "indexes by primary and alternate emails" do
       ml = create(:mailing_list, email: "pgsql-hackers@lists.postgresql.org",
-                                 alternate_emails: ["pgsql-hackers@postgresql.org"])
+                                 alternate_emails: [ "pgsql-hackers@postgresql.org" ])
       index = described_class.email_lookup_index
       expect(index["pgsql-hackers@lists.postgresql.org"]).to eq(ml)
       expect(index["pgsql-hackers@postgresql.org"]).to eq(ml)

spec/services/imap_idle_runner_spec.rb

@@ -57,7 +57,7 @@
       state = ImapSyncState.for_label("INBOX")
       allow(imap_client).to receive(:connect!).and_return(true)
       allow(imap_client).to receive(:disconnect!).and_return(true)
-      expect(imap_client).to receive(:uids_after).with(0).and_return([201])
+      expect(imap_client).to receive(:uids_after).with(0).and_return([ 201 ])
       raw = <<~MAIL
         From: Test <test@example.com>
         To: pgsql-hackers@lists.postgresql.org
@@ -83,11 +83,11 @@
     end
 
     it "resolves list from alternate email in CC" do
-      hackers_list.update!(alternate_emails: ["pgsql-hackers@postgresql.org"])
+      hackers_list.update!(alternate_emails: [ "pgsql-hackers@postgresql.org" ])
       state = ImapSyncState.for_label("INBOX")
       allow(imap_client).to receive(:connect!).and_return(true)
       allow(imap_client).to receive(:disconnect!).and_return(true)
-      expect(imap_client).to receive(:uids_after).with(0).and_return([203])
+      expect(imap_client).to receive(:uids_after).with(0).and_return([ 203 ])
       raw = <<~MAIL
         From: Test <test@example.com>
         To: peter@eisentraut.org
@@ -117,7 +117,7 @@
       state = ImapSyncState.for_label("INBOX")
       allow(imap_client).to receive(:connect!).and_return(true)
       allow(imap_client).to receive(:disconnect!).and_return(true)
-      expect(imap_client).to receive(:uids_after).with(0).and_return([202])
+      expect(imap_client).to receive(:uids_after).with(0).and_return([ 202 ])
       raw = <<~MAIL
         From: Test <test@example.com>
         To: someone@example.com