rubocop

.rubocop.yml

@@ -42,6 +42,10 @@ Style/ClassAndModuleChildren:
 Style/GuardClause:
   Enabled: false
 
+Style/IndentHeredoc:
+  # TODO(glebm): Remove this when we update the minimum Ruby version to 2.3+.
+  Enabled: false
+
 Style/MultilineBlockChain:
   Enabled: false
 

Gemfile

@@ -1,12 +1,13 @@
 # frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in i18n-tasks.gemspec
 gemspec
 
 unless ENV['TRAVIS']
   group :development do
-    gem 'byebug', platforms: [:mri, :mswin, :x64_mingw_21, :x64_mingw_22], require: false
+    gem 'byebug', platforms: %i(mri mswin x64_mingw_21 x64_mingw_22), require: false
     gem 'rubinius-debugger', platform: :rbx, require: false
   end
 end

Rakefile

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:rspec)

i18n-tasks.gemspec

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'i18n/tasks/version'

lib/i18n/tasks.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # define all the modules to be able to use ::
 module I18n
   module Tasks

lib/i18n/tasks/base_task.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command_error'
 require 'i18n/tasks/split_key'
 require 'i18n/tasks/key_pattern_matching'

lib/i18n/tasks/cli.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks'
 require 'i18n/tasks/commands'
 require 'optparse'

lib/i18n/tasks/command/collection.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command/dsl'
 require 'i18n/tasks/command/options/common'
 require 'i18n/tasks/command/options/locales'

lib/i18n/tasks/command/commander.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/cli'
 require 'i18n/tasks/reports/terminal'
 require 'i18n/tasks/reports/spreadsheet'

lib/i18n/tasks/command/commands/data.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands
@@ -13,7 +14,7 @@ module Data
         cmd :normalize,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.normalize'),
-            args: [:locales, :pattern_router]
+            args: %i(locales pattern_router)
 
         def normalize(opt = {})
           i18n.normalize_store! opt[:locales], opt[:pattern_router]
@@ -48,7 +49,7 @@ def rm(opt = {})
         cmd :data,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.data'),
-            args: [:locales, :out_format]
+            args: %i(locales out_format)
 
         def data(opt = {})
           print_forest i18n.data_forest(opt[:locales]), opt
@@ -57,7 +58,7 @@ def data(opt = {})
         cmd :data_merge,
             pos:  '[tree ...]',
             desc: t('i18n_tasks.cmd.desc.data_merge'),
-            args: [:data_format, :nostdin]
+            args: %i(data_format nostdin)
 
         def data_merge(opt = {})
           forest = merge_forests_stdin_and_pos!(opt)
@@ -68,7 +69,7 @@ def data_merge(opt = {})
         cmd :data_write,
             pos:  '[tree]',
             desc: t('i18n_tasks.cmd.desc.data_write'),
-            args: [:data_format, :nostdin]
+            args: %i(data_format nostdin)
 
         def data_write(opt = {})
           forest = forest_pos_or_stdin!(opt)
@@ -79,7 +80,7 @@ def data_write(opt = {})
         cmd :data_remove,
             pos:  '[tree]',
             desc: t('i18n_tasks.cmd.desc.data_remove'),
-            args: [:data_format, :nostdin]
+            args: %i(data_format nostdin)
 
         def data_remove(opt = {})
           removed = i18n.data.remove_by_key!(forest_pos_or_stdin!(opt))

lib/i18n/tasks/command/commands/eq_base.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands
@@ -8,7 +9,7 @@ module EqBase
         cmd :eq_base,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.eq_base'),
-            args: [:locales, :out_format]
+            args: %i(locales out_format)
 
         def eq_base(opt = {})
           print_forest i18n.eq_base_keys(opt), opt, :eq_base_keys

lib/i18n/tasks/command/commands/health.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands
@@ -8,7 +9,7 @@ module Health
         cmd :health,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.health'),
-            args: [:locales, :out_format]
+            args: %i(locales out_format)
 
         def health(opt = {})
           forest = i18n.data_forest(opt[:locales])

lib/i18n/tasks/command/commands/meta.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands

lib/i18n/tasks/command/commands/missing.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command/collection'
 
 module I18n::Tasks
@@ -24,7 +25,7 @@ module Missing
         cmd :missing,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.missing'),
-            args: [:locales, :out_format, :missing_types]
+            args: %i(locales out_format missing_types)
 
         def missing(opt = {})
           forest = i18n.missing_keys(opt.slice(:locales, :base_locale, :missing_types))

lib/i18n/tasks/command/commands/tree.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands
@@ -19,7 +20,7 @@ def tree_translate(opts = {})
         cmd :tree_merge,
             pos:  '[[tree] [tree] ... (or stdin)]',
             desc: t('i18n_tasks.cmd.desc.tree_merge'),
-            args: [:data_format, :nostdin]
+            args: %i(data_format nostdin)
 
         def tree_merge(opts = {})
           print_forest merge_forests_stdin_and_pos!(opts), opts
@@ -28,7 +29,7 @@ def tree_merge(opts = {})
         cmd :tree_filter,
             pos:  '[pattern] [tree (or stdin)]',
             desc: t('i18n_tasks.cmd.desc.tree_filter'),
-            args: [:data_format, :pattern]
+            args: %i(data_format pattern)
 
         def tree_filter(opts = {})
           pattern = arg_or_pos! :pattern, opts
@@ -74,7 +75,7 @@ def tree_mv(opt = {})
         cmd :tree_subtract,
             pos:  '[[tree] [tree] ... (or stdin)]',
             desc: t('i18n_tasks.cmd.desc.tree_subtract'),
-            args: [:data_format, :nostdin]
+            args: %i(data_format nostdin)
 
         def tree_subtract(opt = {})
           forests = forests_stdin_and_pos! opt, 2
@@ -85,7 +86,7 @@ def tree_subtract(opt = {})
         cmd :tree_set_value,
             pos:  '[VALUE] [tree (or stdin)]',
             desc: t('i18n_tasks.cmd.desc.tree_set_value'),
-            args: [:value, :data_format, :nostdin, :pattern]
+            args: %i(value data_format nostdin pattern)
 
         def tree_set_value(opt = {})
           value       = arg_or_pos! :value, opt

lib/i18n/tasks/command/commands/usages.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands
@@ -12,7 +13,7 @@ module Usages
         cmd :find,
             pos:  '[pattern]',
             desc: t('i18n_tasks.cmd.desc.find'),
-            args: [:out_format, :pattern, :strict]
+            args: %i(out_format pattern strict)
 
         def find(opt = {})
           opt[:filter] ||= opt.delete(:pattern) || opt[:arguments].try(:first)
@@ -23,7 +24,7 @@ def find(opt = {})
         cmd :unused,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.unused'),
-            args: [:locales, :out_format, :strict]
+            args: %i(locales out_format strict)
 
         def unused(opt = {})
           forest = i18n.unused_keys(opt.slice(:locales, :strict))
@@ -34,7 +35,7 @@ def unused(opt = {})
         cmd :remove_unused,
             pos:  '[locale ...]',
             desc: t('i18n_tasks.cmd.desc.remove_unused'),
-            args: [:locales, :out_format, :strict, :confirm]
+            args: %i(locales out_format strict confirm)
 
         def remove_unused(opt = {})
           unused_keys = i18n.unused_keys(opt.slice(:locales, :strict))

lib/i18n/tasks/command/commands/xlsx.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module Commands

lib/i18n/tasks/command/dsl.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module DSL

lib/i18n/tasks/command/option_parsers/enum.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module OptionParsers

lib/i18n/tasks/command/option_parsers/locale.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module Command
     module OptionParsers
@@ -39,7 +40,7 @@ def call(vals, context)
             if vals == %w(all) || vals.blank?
               context.locales
             else
-              move_base_locale_to_front! vals.map { |v| v == 'base' ? context.base_locale : v }, context.base_locale
+              move_base_locale_to_front!(vals.map { |v| v == 'base' ? context.base_locale : v }, context.base_locale)
             end.tap do |locales|
               locales.each { |locale| validate! locale }
             end

lib/i18n/tasks/command/options/common.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command/dsl'
 
 module I18n::Tasks

lib/i18n/tasks/command/options/data.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command/option_parsers/enum'
 
 module I18n::Tasks

lib/i18n/tasks/command/options/locales.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command/option_parsers/locale'
 
 module I18n::Tasks

lib/i18n/tasks/command_error.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n
   module Tasks
     # When this type of error is caught:

lib/i18n/tasks/commands.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/command/dsl'
 require 'i18n/tasks/command/collection'
 require 'i18n/tasks/command/commands/health'

lib/i18n/tasks/configuration.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks::Configuration # rubocop:disable Metrics/ModuleLength
   DEFAULTS = {
     base_locale:     'en',

lib/i18n/tasks/console_context.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   class ConsoleContext < BaseTask
     def to_s

lib/i18n/tasks/data.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/data/file_system'
 
 module I18n::Tasks

lib/i18n/tasks/data/adapter/json_adapter.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'json'
 
 module I18n::Tasks

lib/i18n/tasks/data/adapter/yaml_adapter.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'yaml'
 module I18n::Tasks
   module Data

lib/i18n/tasks/data/file_formats.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'fileutils'
 
 module I18n

lib/i18n/tasks/data/file_system.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/data/file_system_base'
 require 'i18n/tasks/data/adapter/json_adapter'
 require 'i18n/tasks/data/adapter/yaml_adapter'

lib/i18n/tasks/data/file_system_base.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/data/tree/node'
 require 'i18n/tasks/data/router/pattern_router'
 require 'i18n/tasks/data/router/conservative_router'

lib/i18n/tasks/data/router/conservative_router.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/data/router/pattern_router'
 
 module I18n::Tasks

lib/i18n/tasks/data/router/pattern_router.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'i18n/tasks/key_pattern_matching'
 require 'i18n/tasks/data/tree/node'
 

lib/i18n/tasks/data/tree/siblings.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'set'
 require 'i18n/tasks/split_key'
 require 'i18n/tasks/data/tree/nodes'

lib/i18n/tasks/data/tree/traversal.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'set'
 
 module I18n::Tasks

lib/i18n/tasks/google_translation.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'easy_translate'
 require 'i18n/tasks/html_keys'
 

lib/i18n/tasks/html_keys.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks
   module HtmlKeys
     HTML_KEY_PATTERN = /[.\-_]html\z/

lib/i18n/tasks/ignore_keys.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module I18n::Tasks::IgnoreKeys
   # whether to ignore the key
   # will also apply global ignore rules