t 2.0.1 → 2.0.2

data/CONTRIBUTING.md

@@ -37,7 +37,7 @@ Ideally, a bug report should include a pull request with failing specs.
 3. Add specs for your unimplemented feature or bug fix.
 4. Run `bundle exec rake spec`. If your specs pass, return to step 3.
 5. Implement your feature or bug fix.
-6. Run `bundle exec rake spec`. If your specs fail, return to step 5.
+6. Run `bundle exec rake default`. If your specs fail, return to step 5.
 7. Run `open coverage/index.html`. If your changes are not completely covered
    by your tests, return to step 3.
 8. Add, commit, and push your changes.

data/README.md

@@ -15,7 +15,7 @@
 [gittip]: https://www.gittip.com/sferik/
 
 #### A command-line power tool for Twitter.
-The CLI takes syntactic cues from the [Twitter SMS commands][sms], however it
+The CLI takes syntactic cues from the [Twitter SMS commands][sms], but it
 offers vastly more commands and capabilities than are available via SMS.
 
 [sms]: https://support.twitter.com/articles/14020-twitter-sms-command
@@ -29,7 +29,7 @@ First, make sure you have Ruby installed.
 
 If the output looks something like this, you're in good shape:
 
-    ruby 1.9.3p385 (2013-02-06 revision 39114) [x86_64-darwin12.2.1]
+    ruby 1.9.3p484 (2013-11-22 revision 43786) [x86_64-darwin13.0.0]
 
 If the output looks more like this, you need to [install Ruby][ruby]:
 [ruby]: http://www.ruby-lang.org/en/downloads/
@@ -59,14 +59,14 @@ public key as a trusted certificate (you only need to do this once):
 
     gem cert --add <(curl -Ls https://raw.github.com/sferik/t/master/certs/sferik.pem)
 
-Then, install the gem with the high security trust policy:
+Then, install the gem with the medium security trust policy:
 
-    gem install t -P HighSecurity
+    gem install t -P MediumSecurity
 
 ## Configuration
 Twitter API v1.1 requires OAuth for all of its functionality, so you'll need a
 registered Twitter application. If you've never registered a Twitter
-application before, it's easy! Just sign-in using your Twitter account and the
+application before, it's easy! Just sign-in using your Twitter account and then
 fill out the short form at <http://dev.twitter.com/apps/new>. If you've
 previously registered a Twitter application, it should be listed at
 <http://dev.twitter.com/apps>. Once you've registered an application, make sure
@@ -138,8 +138,8 @@ obviously can't contain any apostrophes.)
     t does_follow @ev @sferik
 
 **Note**: If the first user does not follow the second, `t` will exit with a
-non-zero exit code. This allows you to execute commands conditionally, for
-example, send a user a direct message only if he already follows you:
+non-zero exit code. This allows you to execute commands conditionally. For
+example, here's how to send a user a direct message only if they already follow you:
 
     t does_follow @ev && t dm @ev "What's up, bro?"
 
@@ -179,13 +179,13 @@ example, send a user a direct message only if he already follows you:
 #### Start streaming your timeline (Control-C to stop)
     t stream timeline
 
-#### Count the number of employees who work for Twitter
-    t list members twitter/team | wc -l
+#### Count the number of official Twitter engineering accounts
+    t list members twitter/engineering | wc -l
 
 #### Search Twitter for the 20 most recent Tweets that match a specified query
     t search all "query"
 
-#### Download the latest Linux kernel via BitTorrent (possibly NSFW, depending where you work)
+#### Download the latest Linux kernel via BitTorrent (possibly NSFW, depending on where you work)
     t search all "lang:en filter:links linux torrent" -n 1 | grep -o "http://t.co/[0-9A-Za-z]*" | xargs open
 
 #### Search Tweets you've favorited that match a specified query
@@ -204,7 +204,7 @@ example, send a user a direct message only if he already follows you:
     t search timeline @sferik "query"
 
 ## Features
-* Deep search: Instead of using the Twitter Search API, [which only only goes
+* Deep search: Instead of using the Twitter Search API, [which only goes
   back 6-9 days][search], `t search` fetches up to 3,200 tweets via the REST API
   and then checks each one against a regular expression.
 * Multi-threaded: Whenever possible, Twitter API requests are made in parallel,
@@ -275,7 +275,7 @@ shell?
 
 ## History
 The [twitter gem][gem] previously contained a command-line interface, up until
-version 0.5.0, when it was [removed][]. This project is offered as a sucessor
+version 0.5.0, when it was [removed][]. This project is offered as a successor
 to that effort, however it is a clean room implementation that contains none of
 the original code.
 

data/Rakefile

@@ -4,8 +4,18 @@ Bundler::GemHelper.install_tasks
 require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:spec)
 
+begin
+  require 'rubocop/rake_task'
+  Rubocop::RakeTask.new
+rescue LoadError
+  desc 'Run RuboCop'
+  task :rubocop do
+    $stderr.puts 'Rubocop is disabled'
+  end
+end
+
 Dir.glob('tasks/*.rake').each { |r| import r }
 
 task :release => 'completion:zsh'
 task :test => :spec
-task :default => :spec
+task :default => [:spec, :rubocop]

data/bin/t

@@ -2,7 +2,7 @@
 
 # Trap interrupts to quit cleanly. See
 # https://twitter.com/mitchellh/status/283014103189053442
-Signal.trap("INT") { exit 1 }
+Signal.trap('INT') { exit 1 }
 
 require 'oauth'
 require 't'
@@ -11,7 +11,7 @@ require 'twitter'
 # Output message to $stderr, prefixed with the program name
 def pute(*args)
   first = args.shift.dup
-  first.insert(0, "#{$0}: ")
+  first.insert(0, "#{$PROGRAM_NAME}: ")
   args.unshift(first)
   $stderr.puts(*args)
 end
@@ -19,29 +19,29 @@ end
 begin
   T::CLI.start(ARGV)
 rescue Interrupt
-  pute "Quitting..."
+  pute 'Quitting...'
   exit 1
 rescue OAuth::Unauthorized
-  pute "Authorization failed"
+  pute 'Authorization failed'
   exit 1
 rescue Twitter::Error::TooManyRequests => error
   pute error.message,
-    "The rate limit for this request will reset in #{error.rate_limit.reset_in} seconds.",
-    "While you wait, consider making a polite request for Twitter to increase the API rate limit at https://dev.twitter.com/discussions/10644"
+       "The rate limit for this request will reset in #{error.rate_limit.reset_in} seconds.",
+       'While you wait, consider making a polite request for Twitter to increase the API rate limit at https://dev.twitter.com/discussions/10644'
   exit 1
 rescue Twitter::Error::BadRequest => error
   pute error.message,
-    "Run `t authorize` to authorize."
+       'Run `t authorize` to authorize.'
   exit 1
 rescue Twitter::Error::Forbidden, Twitter::Error::Unauthorized => error
   pute error.message
-  if error.message == "Error processing your OAuth request: Read-only application cannot POST" ||
-    error.message == "This application is not allowed to access or delete your direct messages"
+  if error.message == 'Error processing your OAuth request: Read-only application cannot POST' ||
+    error.message == 'This application is not allowed to access or delete your direct messages'
     $stderr.puts(%q(Make sure to set your Twitter application's Access Level to "Read, Write and Access direct messages".))
     require 'thor'
-    Thor::Shell::Basic.new.ask "Press [Enter] to open the Twitter Developer site."
+    Thor::Shell::Basic.new.ask 'Press [Enter] to open the Twitter Developer site.'
     require 'launchy'
-    Launchy.open("https://dev.twitter.com/apps") { |u,o,e| $stderr.puts "Manually open #{u}" }
+    Launchy.open('https://dev.twitter.com/apps') { |u, o, e| $stderr.puts "Manually open #{u}" }
   end
   exit 1
 rescue Twitter::Error => error

data/lib/t.rb

@@ -3,7 +3,6 @@ require 'time'
 
 module T
   class << self
-
     # Convert time to local time by applying the `utc_offset` setting.
     def local_time(time)
       utc_offset ? (time.dup.utc + utc_offset) : time.localtime
@@ -15,14 +14,13 @@ module T
 
     def utc_offset=(offset)
       @utc_offset = case offset
-      when String
-        Time.zone_offset(offset)
-      when NilClass
-        nil
-      else
-        offset.to_i
-      end
+                    when String
+                      Time.zone_offset(offset)
+                    when NilClass
+                      nil
+                    else
+                      offset.to_i
+                    end
     end
-
   end
 end

data/lib/t/cli.rb

@@ -24,20 +24,20 @@ module T
     include T::Utils
 
     DEFAULT_NUM_RESULTS = 20
-    DIRECT_MESSAGE_HEADINGS = ["ID", "Posted at", "Screen name", "Text"]
-    TREND_HEADINGS = ["WOEID", "Parent ID", "Type", "Name", "Country"]
+    DIRECT_MESSAGE_HEADINGS = ['ID', 'Posted at', 'Screen name', 'Text']
+    TREND_HEADINGS = ['WOEID', 'Parent ID', 'Type', 'Name', 'Country']
 
     check_unknown_options!
 
-    class_option "color", :aliases => "-C", :type => :string, :enum => %w(auto never), :default => "auto", :desc => "Control how color is used in output"
-    class_option "profile", :aliases => "-P", :type => :string, :default => File.join(File.expand_path("~"), T::RCFile::FILE_NAME), :desc => "Path to RC file", :banner => "FILE"
+    class_option 'color', :aliases => '-C', :type => :string, :enum => %w(auto never), :default => 'auto', :desc => 'Control how color is used in output'
+    class_option 'profile', :aliases => '-P', :type => :string, :default => File.join(File.expand_path('~'), T::RCFile::FILE_NAME), :desc => 'Path to RC file', :banner => 'FILE'
 
     def initialize(*)
       @rcfile = T::RCFile.instance
       super
     end
 
-    desc "accounts", "List accounts"
+    desc 'accounts', 'List accounts'
     def accounts
       @rcfile.path = options['profile'] if options['profile']
       @rcfile.profiles.each do |profile|
@@ -48,53 +48,53 @@ module T
       end
     end
 
-    desc "authorize", "Allows an application to request user authorization"
-    method_option "display-uri", :aliases => "-d", :type => :boolean, :default => false, :desc => "Display the authorization URL instead of attempting to open it."
+    desc 'authorize', 'Allows an application to request user authorization'
+    method_option 'display-uri', :aliases => '-d', :type => :boolean, :default => false, :desc => 'Display the authorization URL instead of attempting to open it.'
     def authorize
       @rcfile.path = options['profile'] if options['profile']
       if @rcfile.empty?
         say "Welcome! Before you can use t, you'll first need to register an"
-        say "application with Twitter. Just follow the steps below:"
-        say "  1. Sign in to the Twitter Developer site and click"
+        say 'application with Twitter. Just follow the steps below:'
+        say '  1. Sign in to the Twitter Developer site and click'
         say "     \"Create a new application\"."
-        say "  2. Complete the required fields and submit the form."
-        say "     Note: Your application must have a unique name."
+        say '  2. Complete the required fields and submit the form.'
+        say '     Note: Your application must have a unique name.'
         say "     We recommend: \"<your handle>/t\"."
-        say "  3. Go to the Settings tab of your application, and change the"
+        say '  3. Go to the Settings tab of your application, and change the'
         say "     Access setting to \"Read, Write and Access direct messages\"."
-        say "  4. Go to the Details tab to view the consumer key and secret,"
+        say '  4. Go to the Details tab to view the consumer key and secret,'
         say "     which you'll need to copy and paste below when prompted."
         say
-        ask "Press [Enter] to open the Twitter Developer site."
+        ask 'Press [Enter] to open the Twitter Developer site.'
         say
       else
         say "It looks like you've already registered an application with Twitter."
-        say "To authorize a new account, just follow the steps below:"
-        say "  1. Sign in to the Twitter Developer site."
+        say 'To authorize a new account, just follow the steps below:'
+        say '  1. Sign in to the Twitter Developer site.'
         say "  2. Select the application for which you'd like to authorize an account."
-        say "  3. Copy and paste the consumer key and secret below when prompted."
+        say '  3. Copy and paste the consumer key and secret below when prompted.'
         say
-        ask "Press [Enter] to open the Twitter Developer site."
+        ask 'Press [Enter] to open the Twitter Developer site.'
         say
       end
       require 'launchy'
-      open_or_print( "https://dev.twitter.com/apps", :dry_run => options['display-uri'] )
-      key = ask "Enter your consumer key:"
-      secret = ask "Enter your consumer secret:"
+      open_or_print('https://dev.twitter.com/apps', :dry_run => options['display-uri'])
+      key = ask 'Enter your consumer key:'
+      secret = ask 'Enter your consumer secret:'
       consumer = OAuth::Consumer.new(key, secret, :site => Twitter::REST::Client::ENDPOINT)
       request_token = consumer.get_request_token
       uri = generate_authorize_uri(consumer, request_token)
       say
-      say "In a moment, you will be directed to the Twitter app authorization page."
-      say "Perform the following steps to complete the authorization process:"
-      say "  1. Sign in to Twitter."
+      say 'In a moment, you will be directed to the Twitter app authorization page.'
+      say 'Perform the following steps to complete the authorization process:'
+      say '  1. Sign in to Twitter.'
       say "  2. Press \"Authorize app\"."
-      say "  3. Copy and paste the supplied PIN below when prompted."
+      say '  3. Copy and paste the supplied PIN below when prompted.'
       say
-      ask "Press [Enter] to open the Twitter app authorization page."
+      ask 'Press [Enter] to open the Twitter app authorization page.'
       say
       open_or_print(uri, :dry_run => options['display-uri'])
-      pin = ask "Enter the supplied PIN:"
+      pin = ask 'Enter the supplied PIN:'
       access_token = request_token.get_access_token(:oauth_verifier => pin.chomp)
       oauth_response = access_token.get('/1.1/account/verify_credentials.json?include_entities=false&skip_status=true')
       screen_name = oauth_response.body.match(/"screen_name"\s*:\s*"(.*?)"/).captures.first
@@ -108,25 +108,25 @@ module T
         }
       }
       @rcfile.active_profile = {'username' => screen_name, 'consumer_key' => key}
-      say "Authorization successful."
+      say 'Authorization successful.'
     end
 
-    desc "block USER [USER...]", "Block users."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify input as Twitter user IDs instead of screen names."
+    desc 'block USER [USER...]', 'Block users.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify input as Twitter user IDs instead of screen names.'
     def block(user, *users)
-      users, number = fetch_users(users.unshift(user), options) do |users|
-        client.block(users)
+      blocked_users, number = fetch_users(users.unshift(user), options) do |users_to_block|
+        client.block(users_to_block)
       end
       say "@#{@rcfile.active_profile[0]} blocked #{pluralize(number, 'user')}."
       say
-      say "Run `#{File.basename($0)} delete block #{users.map{|user| "@#{user.screen_name}"}.join(' ')}` to unblock."
+      say "Run `#{File.basename($PROGRAM_NAME)} delete block #{blocked_users.map { |blocked_user| "@#{blocked_user.screen_name}" }.join(' ')}` to unblock."
     end
 
-    desc "direct_messages", "Returns the #{DEFAULT_NUM_RESULTS} most recent Direct Messages sent to you."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "number", :aliases => "-n", :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => "Limit the number of results."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
+    desc 'direct_messages', "Returns the #{DEFAULT_NUM_RESULTS} most recent Direct Messages sent to you."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'number', :aliases => '-n', :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => 'Limit the number of results.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
     def direct_messages
       count = options['number'] || DEFAULT_NUM_RESULTS
       direct_messages = collect_with_count(count) do |count_opts|
@@ -144,7 +144,7 @@ module T
         array = direct_messages.map do |direct_message|
           [direct_message.id, ls_formatted_time(direct_message), "@#{direct_message.sender.screen_name}", HTMLEntities.new.decode(direct_message.text).gsub(/\n+/, ' ')]
         end
-        format = options['format'] || DIRECT_MESSAGE_HEADINGS.size.times.map{"%s"}
+        format = options['format'] || DIRECT_MESSAGE_HEADINGS.size.times.map { '%s' }
         print_table_with_headings(array, DIRECT_MESSAGE_HEADINGS, format)
       else
         direct_messages.each do |direct_message|
@@ -154,11 +154,11 @@ module T
     end
     map %w(directmessages dms) => :direct_messages
 
-    desc "direct_messages_sent", "Returns the #{DEFAULT_NUM_RESULTS} most recent Direct Messages you've sent."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "number", :aliases => "-n", :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => "Limit the number of results."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
+    desc 'direct_messages_sent', "Returns the #{DEFAULT_NUM_RESULTS} most recent Direct Messages you've sent."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'number', :aliases => '-n', :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => 'Limit the number of results.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
     def direct_messages_sent
       count = options['number'] || DEFAULT_NUM_RESULTS
       direct_messages = collect_with_count(count) do |count_opts|
@@ -176,7 +176,7 @@ module T
         array = direct_messages.map do |direct_message|
           [direct_message.id, ls_formatted_time(direct_message), "@#{direct_message.recipient.screen_name}", HTMLEntities.new.decode(direct_message.text).gsub(/\n+/, ' ')]
         end
-        format = options['format'] || DIRECT_MESSAGE_HEADINGS.size.times.map{"%s"}
+        format = options['format'] || DIRECT_MESSAGE_HEADINGS.size.times.map { '%s' }
         print_table_with_headings(array, DIRECT_MESSAGE_HEADINGS, format)
       else
         direct_messages.each do |direct_message|
@@ -186,24 +186,20 @@ module T
     end
     map %w(directmessagessent sent_messages sentmessages sms) => :direct_messages_sent
 
-    desc "groupies [USER]", "Returns the list of people who follow you but you don't follow back."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => "screen_name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
-    def groupies(user=nil)
+    desc 'groupies [USER]', "Returns the list of people who follow you but you don't follow back."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => 'screen_name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
+    def groupies(user = nil)
       user = if user
-        require 't/core_ext/string'
-        if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
-      else
-        client.verify_credentials.screen_name
-      end
+               require 't/core_ext/string'
+               options['id'] ? user.to_i : user.strip_ats
+             else
+               client.verify_credentials.screen_name
+             end
       follower_ids = Thread.new do
         client.follower_ids(user).to_a
       end
@@ -219,33 +215,25 @@ module T
     end
     map %w(disciples) => :groupies
 
-    desc "dm USER MESSAGE", "Sends that person a Direct Message."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
+    desc 'dm USER MESSAGE', 'Sends that person a Direct Message.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
     def dm(user, message)
       require 't/core_ext/string'
-      user = if options['id']
-        user.to_i
-      else
-        user.strip_ats
-      end
-      direct_message = client.direct_message_create(user, message)
+      user = options['id'] ? user.to_i : user.strip_ats
+      direct_message = client.create_direct_message(user, message)
       say "Direct Message sent from @#{@rcfile.active_profile[0]} to @#{direct_message.recipient.screen_name}."
     end
     map %w(d m) => :dm
 
-    desc "does_contain [USER/]LIST USER", "Find out whether a list contains a user."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    def does_contain(list, user=nil)
+    desc 'does_contain [USER/]LIST USER', 'Find out whether a list contains a user.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    def does_contain(list, user = nil)
       owner, list = extract_owner(list, options)
       if user.nil?
         user = @rcfile.active_profile[0]
       else
         require 't/core_ext/string'
-        user = if options['id']
-          user = client.user(user.to_i).screen_name
-        else
-          user.strip_ats
-        end
+        user = options['id'] ? client.user(user.to_i).screen_name : user.strip_ats
       end
       if client.list_member?(owner, list, user)
         say "Yes, #{list} contains @#{user}."
@@ -256,23 +244,15 @@ module T
     end
     map %w(dc doescontain) => :does_contain
 
-    desc "does_follow USER [USER]", "Find out whether one user follows another."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    def does_follow(user1, user2=nil)
+    desc 'does_follow USER [USER]', 'Find out whether one user follows another.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    def does_follow(user1, user2 = nil)
       require 't/core_ext/string'
-      user1 = if options['id']
-        client.user(user1.to_i).screen_name
-      else
-        user1.strip_ats
-      end
+      user1 = options['id'] ? client.user(user1.to_i).screen_name : user1.strip_ats
       if user2.nil?
         user2 = @rcfile.active_profile[0]
       else
-        user2 = if options['id']
-          client.user(user2.to_i).screen_name
-        else
-          user2.strip_ats
-        end
+        user2 = options['id'] ? client.user(user2.to_i).screen_name : user2.strip_ats
       end
       if client.friendship?(user1, user2)
         say "Yes, @#{user1} follows @#{user2}."
@@ -283,7 +263,7 @@ module T
     end
     map %w(df doesfollow) => :does_follow
 
-    desc "favorite TWEET_ID [TWEET_ID...]", "Marks Tweets as favorites."
+    desc 'favorite TWEET_ID [TWEET_ID...]', 'Marks Tweets as favorites.'
     def favorite(status_id, *status_ids)
       status_ids.unshift(status_id)
       status_ids.map!(&:to_i)
@@ -294,19 +274,19 @@ module T
       number = favorites.length
       say "@#{@rcfile.active_profile[0]} favorited #{pluralize(number, 'tweet')}."
       say
-      say "Run `#{File.basename($0)} delete favorite #{status_ids.join(' ')}` to unfavorite."
+      say "Run `#{File.basename($PROGRAM_NAME)} delete favorite #{status_ids.join(' ')}` to unfavorite."
     end
     map %w(fave favourite) => :favorite
 
-    desc "favorites [USER]", "Returns the #{DEFAULT_NUM_RESULTS} most recent Tweets you favorited."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "max_id", :aliases => "-m", :type => :numeric, :desc => "Returns only the results with an ID less than the specified ID."
-    method_option "number", :aliases => "-n", :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => "Limit the number of results."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "since_id", :aliases => "-s", :type => :numeric, :desc => "Returns only the results with an ID greater than the specified ID."
-    def favorites(user=nil)
+    desc 'favorites [USER]', "Returns the #{DEFAULT_NUM_RESULTS} most recent Tweets you favorited."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'max_id', :aliases => '-m', :type => :numeric, :desc => 'Returns only the results with an ID less than the specified ID.'
+    method_option 'number', :aliases => '-n', :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => 'Limit the number of results.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'since_id', :aliases => '-s', :type => :numeric, :desc => 'Returns only the results with an ID greater than the specified ID.'
+    def favorites(user = nil)
       count = options['number'] || DEFAULT_NUM_RESULTS
       opts = {}
       opts[:exclude_replies] = true if options['exclude'] == 'replies'
@@ -315,11 +295,7 @@ module T
       opts[:since_id] = options['since_id'] if options['since_id']
       if user
         require 't/core_ext/string'
-        user = if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
+        user = options['id'] ? user.to_i : user.strip_ats
       end
       tweets = collect_with_count(count) do |count_opts|
         client.favorites(user, count_opts.merge(opts))
@@ -328,32 +304,28 @@ module T
     end
     map %w(faves favourites) => :favorites
 
-    desc "follow USER [USER...]", "Allows you to start following users."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify input as Twitter user IDs instead of screen names."
+    desc 'follow USER [USER...]', 'Allows you to start following users.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify input as Twitter user IDs instead of screen names.'
     def follow(user, *users)
-      users, number = fetch_users(users.unshift(user), options) do |users|
-        client.follow(users)
+      followed_users, number = fetch_users(users.unshift(user), options) do |users_to_follow|
+        client.follow(users_to_follow)
       end
       say "@#{@rcfile.active_profile[0]} is now following #{pluralize(number, 'more user')}."
       say
-      say "Run `#{File.basename($0)} unfollow #{users.map{|user| "@#{user.screen_name}"}.join(' ')}` to stop."
+      say "Run `#{File.basename($PROGRAM_NAME)} unfollow #{followed_users.map { |followed_user| "@#{followed_user.screen_name}" }.join(' ')}` to stop."
     end
 
-    desc "followings [USER]", "Returns a list of the people you follow on Twitter."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => "screen_name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
-    def followings(user=nil)
+    desc 'followings [USER]', 'Returns a list of the people you follow on Twitter.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => 'screen_name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
+    def followings(user = nil)
       if user
         require 't/core_ext/string'
-        user = if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
+        user = options['id'] ? user.to_i : user.strip_ats
       end
       following_ids = client.friend_ids(user).to_a
       require 'retryable'
@@ -363,21 +335,17 @@ module T
       print_users(users)
     end
 
-    desc "followers [USER]", "Returns a list of the people who follow you on Twitter."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => "screen_name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
-    def followers(user=nil)
+    desc 'followers [USER]', 'Returns a list of the people who follow you on Twitter.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => 'screen_name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
+    def followers(user = nil)
       if user
         require 't/core_ext/string'
-        user = if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
+        user = options['id'] ? user.to_i : user.strip_ats
       end
       follower_ids = client.follower_ids(user).to_a
       require 'retryable'
@@ -387,24 +355,20 @@ module T
       print_users(users)
     end
 
-    desc "friends [USER]", "Returns the list of people who you follow and follow you back."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => "screen_name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
-    def friends(user=nil)
+    desc 'friends [USER]', 'Returns the list of people who you follow and follow you back.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => 'screen_name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
+    def friends(user = nil)
       user = if user
-        require 't/core_ext/string'
-        if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
-      else
-        client.verify_credentials.screen_name
-      end
+               require 't/core_ext/string'
+               options['id'] ? user.to_i : user.strip_ats
+             else
+               client.verify_credentials.screen_name
+             end
       following_ids = Thread.new do
         client.friend_ids(user).to_a
       end
@@ -419,24 +383,20 @@ module T
       print_users(users)
     end
 
-    desc "leaders [USER]", "Returns the list of people who you follow but don't follow you back."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => "screen_name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
-    def leaders(user=nil)
+    desc 'leaders [USER]', "Returns the list of people who you follow but don't follow you back."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => 'screen_name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
+    def leaders(user = nil)
       user = if user
-        require 't/core_ext/string'
-        if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
-      else
-        client.verify_credentials.screen_name
-      end
+               require 't/core_ext/string'
+               options['id'] ? user.to_i : user.strip_ats
+             else
+               client.verify_credentials.screen_name
+             end
       following_ids = Thread.new do
         client.friend_ids(user).to_a
       end
@@ -451,36 +411,32 @@ module T
       print_users(users)
     end
 
-    desc "lists [USER]", "Returns the lists created by a user."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(members mode posted slug subscribers), :default => "slug", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
-    def lists(user=nil)
+    desc 'lists [USER]', 'Returns the lists created by a user.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(members mode since slug subscribers), :default => 'slug', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
+    def lists(user = nil)
       lists = if user
-        require 't/core_ext/string'
-        user = if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
-        client.lists(user)
-      else
-        client.lists
-      end
+                require 't/core_ext/string'
+                user = options['id'] ? user.to_i : user.strip_ats
+                client.lists(user)
+              else
+                client.lists
+              end
       print_lists(lists)
     end
 
-    desc "matrix", "Unfortunately, no one can be told what the Matrix is. You have to see it for yourself."
-    def_delegator :"T::Stream.new", :matrix
+    desc 'matrix', 'Unfortunately, no one can be told what the Matrix is. You have to see it for yourself.'
+    def_delegator :"T::Stream.new", :matrix # rubocop:disable SymbolName
 
-    desc "mentions", "Returns the #{DEFAULT_NUM_RESULTS} most recent Tweets mentioning you."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "number", :aliases => "-n", :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => "Limit the number of results."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
+    desc 'mentions', "Returns the #{DEFAULT_NUM_RESULTS} most recent Tweets mentioning you."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'number', :aliases => '-n', :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => 'Limit the number of results.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
     def mentions
       count = options['number'] || DEFAULT_NUM_RESULTS
       tweets = collect_with_count(count) do |count_opts|
@@ -490,10 +446,10 @@ module T
     end
     map %w(replies) => :mentions
 
-    desc "open USER", "Opens that user's profile in a web browser."
-    method_option "display-uri", :aliases => "-d", :type => :boolean, :default => false, :desc => "Display the requested URL instead of attempting to open it."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "status", :aliases => "-s", :type => :boolean, :default => false, :desc => "Specify input as a Twitter status ID instead of a screen name."
+    desc 'open USER', "Opens that user's profile in a web browser."
+    method_option 'display-uri', :aliases => '-d', :type => :boolean, :default => false, :desc => 'Display the requested URL instead of attempting to open it.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'status', :aliases => '-s', :type => :boolean, :default => false, :desc => 'Specify input as a Twitter status ID instead of a screen name.'
     def open(user)
       require 'launchy'
       if options['id']
@@ -508,9 +464,9 @@ module T
       end
     end
 
-    desc "reply TWEET_ID MESSAGE", "Post your Tweet as a reply directed at another person."
-    method_option "all", :aliases => "-a", :type => :boolean, :default => false, :desc => "Reply to all users mentioned in the Tweet."
-    method_option "location", :aliases => "-l", :type => :string, :default => "location", :desc => "Add location information. If the optional 'latitude,longitude' parameter is not supplied, looks up location by IP address."
+    desc 'reply TWEET_ID MESSAGE', 'Post your Tweet as a reply directed at another person.'
+    method_option 'all', :aliases => '-a', :type => :boolean, :default => false, :desc => 'Reply to all users mentioned in the Tweet.'
+    method_option 'location', :aliases => '-l', :type => :string, :default => 'location', :desc => "Add location information. If the optional 'latitude,longitude' parameter is not supplied, looks up location by IP address."
     def reply(status_id, message)
       status = client.status(status_id.to_i, :include_my_retweet => false)
       users = Array(status.user.screen_name)
@@ -525,20 +481,20 @@ module T
       reply = client.update("#{users.join(' ')} #{message}", opts)
       say "Reply posted by @#{@rcfile.active_profile[0]} to #{users.join(' ')}."
       say
-      say "Run `#{File.basename($0)} delete status #{reply.id}` to delete."
+      say "Run `#{File.basename($PROGRAM_NAME)} delete status #{reply.id}` to delete."
     end
 
-    desc "report_spam USER [USER...]", "Report users for spam."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify input as Twitter user IDs instead of screen names."
+    desc 'report_spam USER [USER...]', 'Report users for spam.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify input as Twitter user IDs instead of screen names.'
     def report_spam(user, *users)
-      users, number = fetch_users(users.unshift(user), options) do |users|
-        client.report_spam(users)
+      _, number = fetch_users(users.unshift(user), options) do |users_to_report|
+        client.report_spam(users_to_report)
       end
       say "@#{@rcfile.active_profile[0]} reported #{pluralize(number, 'user')}."
     end
     map %w(report reportspam spam) => :report_spam
 
-    desc "retweet TWEET_ID [TWEET_ID...]", "Sends Tweets to your followers."
+    desc 'retweet TWEET_ID [TWEET_ID...]', 'Sends Tweets to your followers.'
     def retweet(status_id, *status_ids)
       status_ids.unshift(status_id)
       status_ids.map!(&:to_i)
@@ -549,99 +505,95 @@ module T
       number = retweets.length
       say "@#{@rcfile.active_profile[0]} retweeted #{pluralize(number, 'tweet')}."
       say
-      say "Run `#{File.basename($0)} delete status #{retweets.map{|tweet| tweet.retweeted_status.id}.join(' ')}` to undo."
+      say "Run `#{File.basename($PROGRAM_NAME)} delete status #{retweets.map { |tweet| tweet.retweeted_status.id }.join(' ')}` to undo."
     end
     map %w(rt) => :retweet
 
-    desc "retweets [USER]", "Returns the #{DEFAULT_NUM_RESULTS} most recent Retweets by a user."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "number", :aliases => "-n", :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => "Limit the number of results."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    def retweets(user=nil)
+    desc 'retweets [USER]', "Returns the #{DEFAULT_NUM_RESULTS} most recent Retweets by a user."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'number', :aliases => '-n', :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => 'Limit the number of results.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    def retweets(user = nil)
       count = options['number'] || DEFAULT_NUM_RESULTS
       tweets = if user
-        require 't/core_ext/string'
-        user = if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
-        collect_with_count(count) do |count_opts|
-          client.retweeted_by_user(user, count_opts)
-        end
-      else
-        collect_with_count(count) do |count_opts|
-          client.retweeted_by_me(count_opts)
-        end
-      end
+                 require 't/core_ext/string'
+                 user = options['id'] ? user.to_i : user.strip_ats
+                 collect_with_count(count) do |count_opts|
+                   client.retweeted_by_user(user, count_opts)
+                 end
+               else
+                 collect_with_count(count) do |count_opts|
+                   client.retweeted_by_me(count_opts)
+                 end
+               end
       print_tweets(tweets)
     end
     map %w(rts) => :retweets
 
-    desc "ruler", "Prints a 140-character ruler"
-    method_option "indent", :aliases => "-i", :type => :numeric, :default => 0, :desc => "The number of space to print before the ruler."
+    desc 'ruler', 'Prints a 140-character ruler'
+    method_option 'indent', :aliases => '-i', :type => :numeric, :default => 0, :desc => 'The number of space to print before the ruler.'
     def ruler
       markings = '----|'.chars.cycle.take(140).join
       say "#{' ' * options['indent'].to_i}#{markings}"
     end
 
-    desc "status TWEET_ID", "Retrieves detailed information about a Tweet."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    def status(status_id)
+    desc 'status TWEET_ID', 'Retrieves detailed information about a Tweet.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    def status(status_id) # rubocop:disable CyclomaticComplexity
       status = client.status(status_id.to_i, :include_my_retweet => false)
       location = if status.place?
-        if status.place.name && status.place.attributes && status.place.attributes[:street_address] && status.place.attributes[:locality] && status.place.attributes[:region] && status.place.country
-          [status.place.name, status.place.attributes[:street_address], status.place.attributes[:locality], status.place.attributes[:region], status.place.country].join(", ")
-        elsif status.place.name && status.place.attributes && status.place.attributes[:locality] && status.place.attributes[:region] && status.place.country
-          [status.place.name, status.place.attributes[:locality], status.place.attributes[:region], status.place.country].join(", ")
-        elsif status.place.full_name && status.place.attributes && status.place.attributes[:region] && status.place.country
-          [status.place.full_name, status.place.attributes[:region], status.place.country].join(", ")
-        elsif status.place.full_name && status.place.country
-          [status.place.full_name, status.place.country].join(", ")
-        elsif status.place.full_name
-          status.place.full_name
-        else
-          status.place.name
-        end
-      elsif status.geo?
-        reverse_geocode(status.geo)
-      end
-      status_headings = ["ID", "Posted at", "Screen name", "Text", "Retweets", "Favorites", "Source", "Location"]
+                   if status.place.name && status.place.attributes && status.place.attributes[:street_address] && status.place.attributes[:locality] && status.place.attributes[:region] && status.place.country
+                     [status.place.name, status.place.attributes[:street_address], status.place.attributes[:locality], status.place.attributes[:region], status.place.country].join(', ')
+                   elsif status.place.name && status.place.attributes && status.place.attributes[:locality] && status.place.attributes[:region] && status.place.country
+                     [status.place.name, status.place.attributes[:locality], status.place.attributes[:region], status.place.country].join(', ')
+                   elsif status.place.full_name && status.place.attributes && status.place.attributes[:region] && status.place.country
+                     [status.place.full_name, status.place.attributes[:region], status.place.country].join(', ')
+                   elsif status.place.full_name && status.place.country
+                     [status.place.full_name, status.place.country].join(', ')
+                   elsif status.place.full_name
+                     status.place.full_name
+                   else
+                     status.place.name
+                   end
+                 elsif status.geo?
+                   reverse_geocode(status.geo)
+                 end
+      status_headings = ['ID', 'Posted at', 'Screen name', 'Text', 'Retweets', 'Favorites', 'Source', 'Location']
       if options['csv']
         require 'csv'
         say status_headings.to_csv
         say [status.id, csv_formatted_time(status), status.user.screen_name, decode_full_text(status), status.retweet_count, status.favorite_count, strip_tags(status.source), location].to_csv
       elsif options['long']
         array = [status.id, ls_formatted_time(status), "@#{status.user.screen_name}", decode_full_text(status).gsub(/\n+/, ' '), status.retweet_count, status.favorite_count, strip_tags(status.source), location]
-        format = options['format'] || status_headings.size.times.map{"%s"}
+        format = options['format'] || status_headings.size.times.map { '%s' }
         print_table_with_headings([array], status_headings, format)
       else
         array = []
-        array << ["ID", status.id.to_s]
-        array << ["Text", decode_full_text(status).gsub(/\n+/, ' ')]
-        array << ["Screen name", "@#{status.user.screen_name}"]
-        array << ["Posted at", "#{ls_formatted_time(status)} (#{time_ago_in_words(status.created_at)} ago)"]
-        array << ["Retweets", number_with_delimiter(status.retweet_count)]
-        array << ["Favorites", number_with_delimiter(status.favorite_count)]
-        array << ["Source", strip_tags(status.source)]
-        array << ["Location", location] unless location.nil?
+        array << ['ID', status.id.to_s]
+        array << ['Text', decode_full_text(status).gsub(/\n+/, ' ')]
+        array << ['Screen name', "@#{status.user.screen_name}"]
+        array << ['Posted at', "#{ls_formatted_time(status)} (#{time_ago_in_words(status.created_at)} ago)"]
+        array << ['Retweets', number_with_delimiter(status.retweet_count)]
+        array << ['Favorites', number_with_delimiter(status.favorite_count)]
+        array << ['Source', strip_tags(status.source)]
+        array << ['Location', location] unless location.nil?
         print_table(array)
       end
     end
 
-    desc "timeline [USER]", "Returns the #{DEFAULT_NUM_RESULTS} most recent Tweets posted by a user."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "exclude", :aliases => "-e", :type => :string, :enum => %w(replies retweets), :desc => "Exclude certain types of Tweets from the results.", :banner => "TYPE"
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "max_id", :aliases => "-m", :type => :numeric, :desc => "Returns only the results with an ID less than the specified ID."
-    method_option "number", :aliases => "-n", :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => "Limit the number of results."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "since_id", :aliases => "-s", :type => :numeric, :desc => "Returns only the results with an ID greater than the specified ID."
-    def timeline(user=nil)
+    desc 'timeline [USER]', "Returns the #{DEFAULT_NUM_RESULTS} most recent Tweets posted by a user."
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'exclude', :aliases => '-e', :type => :string, :enum => %w(replies retweets), :desc => 'Exclude certain types of Tweets from the results.', :banner => 'TYPE'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'max_id', :aliases => '-m', :type => :numeric, :desc => 'Returns only the results with an ID less than the specified ID.'
+    method_option 'number', :aliases => '-n', :type => :numeric, :default => DEFAULT_NUM_RESULTS, :desc => 'Limit the number of results.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'since_id', :aliases => '-s', :type => :numeric, :desc => 'Returns only the results with an ID greater than the specified ID.'
+    def timeline(user = nil)
       count = options['number'] || DEFAULT_NUM_RESULTS
       opts = {}
       opts[:exclude_replies] = true if options['exclude'] == 'replies'
@@ -650,11 +602,7 @@ module T
       opts[:since_id] = options['since_id'] if options['since_id']
       if user
         require 't/core_ext/string'
-        user = if options['id']
-          user.to_i
-        else
-          user.strip_ats
-        end
+        user = options['id'] ? user.to_i : user.strip_ats
         tweets = collect_with_count(count) do |count_opts|
           client.user_timeline(user, count_opts.merge(opts))
         end
@@ -667,35 +615,35 @@ module T
     end
     map %w(tl) => :timeline
 
-    desc "trends [WOEID]", "Returns the top 10 trending topics."
-    method_option "exclude-hashtags", :aliases => "-x", :type => :boolean, :default => false, :desc => "Remove all hashtags from the trends list."
-    def trends(woe_id=1)
+    desc 'trends [WOEID]', 'Returns the top 10 trending topics.'
+    method_option 'exclude-hashtags', :aliases => '-x', :type => :boolean, :default => false, :desc => 'Remove all hashtags from the trends list.'
+    def trends(woe_id = 1)
       opts = {}
-      opts.merge!(:exclude => "hashtags") if options['exclude-hashtags']
+      opts.merge!(:exclude => 'hashtags') if options['exclude-hashtags']
       trends = client.trends(woe_id, opts)
       print_attribute(trends, :name)
     end
 
-    desc "trend_locations", "Returns the locations for which Twitter has trending topic information."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(country name parent type woeid), :default => "name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
+    desc 'trend_locations', 'Returns the locations for which Twitter has trending topic information.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(country name parent type woeid), :default => 'name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
     def trend_locations
       places = client.trend_locations
       places = case options['sort']
-      when 'country'
-        places.sort_by{|places| places.country.downcase}
-      when 'parent'
-        places.sort_by{|places| places.parent_id.to_i}
-      when 'type'
-        places.sort_by{|places| places.place_type.downcase}
-      when 'woeid'
-        places.sort_by{|places| places.woeid.to_i}
-      else
-        places.sort_by{|places| places.name.downcase}
-      end unless options['unsorted']
+               when 'country'
+                 places.sort_by { |place| place.country.downcase }
+               when 'parent'
+                 places.sort_by { |place| place.parent_id.to_i }
+               when 'type'
+                 places.sort_by { |place| place.place_type.downcase }
+               when 'woeid'
+                 places.sort_by { |place| place.woeid.to_i }
+               else
+                 places.sort_by { |place| place.name.downcase }
+               end unless options['unsorted']
       places.reverse! if options['reverse']
       if options['csv']
         require 'csv'
@@ -707,7 +655,7 @@ module T
         array = places.map do |place|
           [place.woeid, place.parent_id, place.place_type, place.name, place.country]
         end
-        format = options['format'] || TREND_HEADINGS.size.times.map{"%s"}
+        format = options['format'] || TREND_HEADINGS.size.times.map { '%s' }
         print_table_with_headings(array, TREND_HEADINGS, format)
       else
         print_attribute(places, :name)
@@ -715,110 +663,102 @@ module T
     end
     map %w(locations trendlocations) => :trend_locations
 
-    desc "unfollow USER [USER...]", "Allows you to stop following users."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify input as Twitter user IDs instead of screen names."
+    desc 'unfollow USER [USER...]', 'Allows you to stop following users.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify input as Twitter user IDs instead of screen names.'
     def unfollow(user, *users)
-      users, number = fetch_users(users.unshift(user), options) do |users|
-        client.unfollow(users)
+      unfollowed_users, number = fetch_users(users.unshift(user), options) do |users_to_unfollow|
+        client.unfollow(users_to_unfollow)
       end
       say "@#{@rcfile.active_profile[0]} is no longer following #{pluralize(number, 'user')}."
       say
-      say "Run `#{File.basename($0)} follow #{users.map{|user| "@#{user.screen_name}"}.join(' ')}` to follow again."
+      say "Run `#{File.basename($PROGRAM_NAME)} follow #{unfollowed_users.map { |unfollowed_user| "@#{unfollowed_user.screen_name}" }.join(' ')}` to follow again."
     end
 
-    desc "update [MESSAGE]", "Post a Tweet."
-    method_option "location", :aliases => "-l", :type => :string, :default => "location", :desc => "Add location information. If the optional 'latitude,longitude' parameter is not supplied, looks up location by IP address."
-    method_option "file", :aliases => "-f", :type => :string, :desc => "The path to an image to attach to your tweet."
-    def update(message=nil)
+    desc 'update [MESSAGE]', 'Post a Tweet.'
+    method_option 'location', :aliases => '-l', :type => :string, :default => 'location', :desc => "Add location information. If the optional 'latitude,longitude' parameter is not supplied, looks up location by IP address."
+    method_option 'file', :aliases => '-f', :type => :string, :desc => 'The path to an image to attach to your tweet.'
+    def update(message = nil)
       message = T::Editor.gets if message.nil? || message.empty?
       opts = {:trim_user => true}
       add_location!(options, opts)
       status = if options['file']
-        client.update_with_media(message, File.new(File.expand_path(options['file'])), opts)
-      else
-        client.update(message, opts)
-      end
+                 client.update_with_media(message, File.new(File.expand_path(options['file'])), opts)
+               else
+                 client.update(message, opts)
+               end
       say "Tweet posted by @#{@rcfile.active_profile[0]}."
       say
-      say "Run `#{File.basename($0)} delete status #{status.id}` to delete."
+      say "Run `#{File.basename($PROGRAM_NAME)} delete status #{status.id}` to delete."
     end
     map %w(post tweet) => :update
 
-    desc "users USER [USER...]", "Returns a list of users you specify."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify input as Twitter user IDs instead of screen names."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
-    method_option "reverse", :aliases => "-r", :type => :boolean, :default => false, :desc => "Reverse the order of the sort."
-    method_option "sort", :aliases => "-s", :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => "screen_name", :desc => "Specify the order of the results.", :banner => "ORDER"
-    method_option "unsorted", :aliases => "-u", :type => :boolean, :default => false, :desc => "Output is not sorted."
+    desc 'users USER [USER...]', 'Returns a list of users you specify.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify input as Twitter user IDs instead of screen names.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
+    method_option 'reverse', :aliases => '-r', :type => :boolean, :default => false, :desc => 'Reverse the order of the sort.'
+    method_option 'sort', :aliases => '-s', :type => :string, :enum => %w(favorites followers friends listed screen_name since tweets tweeted), :default => 'screen_name', :desc => 'Specify the order of the results.', :banner => 'ORDER'
+    method_option 'unsorted', :aliases => '-u', :type => :boolean, :default => false, :desc => 'Output is not sorted.'
     def users(user, *users)
       users.unshift(user)
       require 't/core_ext/string'
-      if options['id']
-        users.map!(&:to_i)
-      else
-        users.map!(&:strip_ats)
-      end
+      options['id'] ? users.map!(&:to_i) : users.map!(&:strip_ats)
       users = client.users(users)
       print_users(users)
     end
     map %w(stats) => :users
 
-    desc "version", "Show version."
+    desc 'version', 'Show version.'
     def version
       require 't/version'
       say T::Version
     end
     map %w(-v --version) => :version
 
-    desc "whois USER", "Retrieves profile information for the user."
-    method_option "csv", :aliases => "-c", :type => :boolean, :default => false, :desc => "Output in CSV format."
-    method_option "id", :aliases => "-i", :type => :boolean, :default => false, :desc => "Specify user via ID instead of screen name."
-    method_option "long", :aliases => "-l", :type => :boolean, :default => false, :desc => "Output in long format."
+    desc 'whois USER', 'Retrieves profile information for the user.'
+    method_option 'csv', :aliases => '-c', :type => :boolean, :default => false, :desc => 'Output in CSV format.'
+    method_option 'id', :aliases => '-i', :type => :boolean, :default => false, :desc => 'Specify user via ID instead of screen name.'
+    method_option 'long', :aliases => '-l', :type => :boolean, :default => false, :desc => 'Output in long format.'
     def whois(user)
       require 't/core_ext/string'
-      user = if options['id']
-        user.to_i
-      else
-        user.strip_ats
-      end
+      user = options['id'] ? user.to_i : user.strip_ats
       user = client.user(user)
       require 'htmlentities'
       if options['csv'] || options['long']
         print_users([user])
       else
         array = []
-        array << ["ID", user.id.to_s]
-        array << ["Since", "#{ls_formatted_time(user)} (#{time_ago_in_words(user.created_at)} ago)"]
-        array << ["Last update", "#{decode_full_text(user.status).gsub(/\n+/, ' ')} (#{time_ago_in_words(user.status.created_at)} ago)"] unless user.status.nil?
-        array << ["Screen name", "@#{user.screen_name}"]
-        array << [user.verified ? "Name (Verified)" : "Name", user.name] unless user.name.nil?
-        array << ["Tweets", number_with_delimiter(user.statuses_count)]
-        array << ["Favorites", number_with_delimiter(user.favorites_count)]
-        array << ["Listed", number_with_delimiter(user.listed_count)]
-        array << ["Following", number_with_delimiter(user.friends_count)]
-        array << ["Followers", number_with_delimiter(user.followers_count)]
-        array << ["Bio", user.description.gsub(/\n+/, ' ')] unless user.description.nil?
-        array << ["Location", user.location] unless user.location.nil?
-        array << ["URL", user.website] unless user.website.nil?
+        array << ['ID', user.id.to_s]
+        array << ['Since', "#{ls_formatted_time(user)} (#{time_ago_in_words(user.created_at)} ago)"]
+        array << ['Last update', "#{decode_full_text(user.status).gsub(/\n+/, ' ')} (#{time_ago_in_words(user.status.created_at)} ago)"] unless user.status.nil?
+        array << ['Screen name', "@#{user.screen_name}"]
+        array << [user.verified ? 'Name (Verified)' : 'Name', user.name] unless user.name.nil?
+        array << ['Tweets', number_with_delimiter(user.statuses_count)]
+        array << ['Favorites', number_with_delimiter(user.favorites_count)]
+        array << ['Listed', number_with_delimiter(user.listed_count)]
+        array << ['Following', number_with_delimiter(user.friends_count)]
+        array << ['Followers', number_with_delimiter(user.followers_count)]
+        array << ['Bio', user.description.gsub(/\n+/, ' ')] unless user.description.nil?
+        array << ['Location', user.location] unless user.location.nil?
+        array << ['URL', user.website] unless user.website.nil?
         print_table(array)
       end
     end
     map %w(user) => :whois
 
-    desc "delete SUBCOMMAND ...ARGS", "Delete Tweets, Direct Messages, etc."
+    desc 'delete SUBCOMMAND ...ARGS', 'Delete Tweets, Direct Messages, etc.'
     subcommand 'delete', T::Delete
 
-    desc "list SUBCOMMAND ...ARGS", "Do various things with lists."
+    desc 'list SUBCOMMAND ...ARGS', 'Do various things with lists.'
     subcommand 'list', T::List
 
-    desc "search SUBCOMMAND ...ARGS", "Search through Tweets."
+    desc 'search SUBCOMMAND ...ARGS', 'Search through Tweets.'
     subcommand 'search', T::Search
 
-    desc "set SUBCOMMAND ...ARGS", "Change various account settings."
+    desc 'set SUBCOMMAND ...ARGS', 'Change various account settings.'
     subcommand 'set', T::Set
 
-    desc "stream SUBCOMMAND ...ARGS", "Commands for streaming Tweets."
+    desc 'stream SUBCOMMAND ...ARGS', 'Commands for streaming Tweets.'
     subcommand 'stream', T::Stream
 
   private
@@ -844,7 +784,7 @@ module T
       params = request['Authorization'].sub(/^OAuth\s+/, '').split(/,\s+/).map do |param|
         key, value = param.split('=')
         value =~ /"(.*?)"/
-        "#{key}=#{CGI::escape($1)}"
+        "#{key}=#{CGI.escape(Regexp.last_match[1])}"
       end.join('&')
       "#{Twitter::REST::Client::ENDPOINT}#{request.path}?#{params}"
     end
@@ -864,7 +804,7 @@ module T
       return @location if @location
       require 'geokit'
       require 'open-uri'
-      ip_address = Kernel::open("http://checkip.dyndns.org/") do |body|
+      ip_address = Kernel.open('http://checkip.dyndns.org/') do |body|
         /(?:\d{1,3}\.){3}\d{1,3}/.match(body.read)[0]
       end
       @location = Geokit::Geocoders::MultiGeocoder.geocode(ip_address)
@@ -874,13 +814,12 @@ module T
       require 'geokit'
       geoloc = Geokit::Geocoders::MultiGeocoder.reverse_geocode(geo.coordinates)
       if geoloc.city && geoloc.state && geoloc.country
-        [geoloc.city, geoloc.state, geoloc.country].join(", ")
+        [geoloc.city, geoloc.state, geoloc.country].join(', ')
       elsif geoloc.state && geoloc.country
-        [geoloc.state, geoloc.country].join(", ")
+        [geoloc.state, geoloc.country].join(', ')
       else
         geoloc.country
       end
     end
-
   end
 end