i18n-tasks 0.9.6 → 0.9.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e38c549e1d0303125dde7b029aa4ed4f5daa13ba
-  data.tar.gz: 55337517009dfeba3b60027c9c2fe1d9dd3aca21
+  metadata.gz: cec4fddc37a4bbc56edf707c49e247e8418a8199
+  data.tar.gz: 38edd490aaa36f481f2c0ae7868649c87ebad8ef
 SHA512:
-  metadata.gz: d9bada34e186461d3c9941f24fba746e06628681d68f925e34a874769c8046cec65d7ed06717fe0f16efefa85543db799c8a8f0997eff2f28ec2f8b3f7b81d44
-  data.tar.gz: e83e89d336812b51fc0ea49fe1364f086d1efa56008e26810955205032d7839d590d905985ef4891bd477e111f1820807f707adfc73d1f33e5245c427ec5f3c7
+  metadata.gz: 2a727430cb2f8ca4ad602db64766da00f6eb9711be9a1425096c18dff6906c60b8af52defdecb67a54521ec7edd18bb7a7487d093aebabbebf5f3e8cc6a58dfc
+  data.tar.gz: 2ec48ffcff555fbb597deed2270a0b0c5d2a859d29069751ff681429292c6652be5c57106a9a5ab22326bce32183cfb1ea36aaeaf70f20ea4303e74330a7fbcf

data/README.md

@@ -24,7 +24,7 @@ i18n-tasks can be used with any project using the ruby [i18n gem][i18n-gem] (def
 Add i18n-tasks to the Gemfile:
 
 ```ruby
-gem 'i18n-tasks', '~> 0.9.6'
+gem 'i18n-tasks', '~> 0.9.7'
 ```
 
 Copy the default [configuration file](#configuration):

data/Rakefile

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:rspec)
-task :default => :rspec
+task default: :rspec
 
 task :irb do
   # $: << File.expand_path('lib', __FILE__)

data/bin/i18n-tasks

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 if ENV['I18N_TASKS_BIN_SIMPLECOV_COVERAGE']
   require_relative '../spec/bin_simplecov_helper'

data/config/locales/en.yml

@@ -21,9 +21,12 @@ en:
           nostdin: Do not read from stdin
           out_format: 'Output format: %{valid_text}'
           pattern_router: 'Use pattern router: keys moved per config data.write'
-          strict: Avoid inferring dynamic key usages such as t("cats.#{cat}.name"). Takes precedence over the
-            config setting if set.
-          value: 'Value. Interpolates: %{value}, %{human_key}, %{key}, %{default}, %{value_or_human_key}, %{value_or_default_or_human_key}'
+          strict: >-
+            Avoid inferring dynamic key usages such as t("cats.#{cat}.name"). Takes precedence over
+            the config setting if set.
+          value: >-
+            Value. Interpolates: %{value}, %{human_key}, %{key}, %{default}, %{value_or_human_key},
+            %{value_or_default_or_human_key}
       desc:
         add_missing: add missing keys to locale data
         config: display i18n-tasks configuration
@@ -73,17 +76,20 @@ en:
       type: Type
       value: Value
     data_stats:
-      text: has %{key_count} keys across %{locale_count} locales. On average, values are %{value_chars_avg}
+      text: >-
+        has %{key_count} keys across %{locale_count} locales. On average, values are %{value_chars_avg}
         characters long, keys have %{key_segments_avg} segments, a locale has %{per_locale_avg} keys.
-      text_single_locale: has %{key_count} keys in total. On average, values are %{value_chars_avg}
-        characters long, keys have %{key_segments_avg} segments.
+      text_single_locale: >-
+        has %{key_count} keys in total. On average, values are %{value_chars_avg} characters long,
+        keys have %{key_segments_avg} segments.
       title: Forest (%{locales})
     google_translate:
       errors:
-        no_api_key: Set Google API key via GOOGLE_TRANSLATE_API_KEY environment variable or translation.api_key
+        no_api_key: >-
+          Set Google API key via GOOGLE_TRANSLATE_API_KEY environment variable or translation.api_key
           in config/i18n-tasks.yml. Get the key at https://code.google.com/apis/console.
-        no_results: Google Translate returned no results. Make sure billing information is set at
-          https://code.google.com/apis/console.
+        no_results: >-
+          Google Translate returned no results. Make sure billing information is set at https://code.google.com/apis/console.
     health:
       no_keys_detected: No keys detected. Check data.read in config/i18n-tasks.yml.
     missing:

data/config/locales/ru.yml

@@ -2,100 +2,106 @@
 ru:
   i18n_tasks:
     add_missing:
-      added: "Добавлены ключи (%{count})"
+      added: Добавлены ключи (%{count})
     cmd:
       args:
-        default_text: "По умолчанию: %{value}"
+        default_text: 'По умолчанию: %{value}'
         desc:
-          confirm: "Подтвердить автоматом"
-          data_format: "Формат данных: %{valid_text}. %{default_text}."
-          key_pattern: "Маска ключа (например, common.*)"
-          key_pattern_to_rename: "Полный ключ (шаблон) для переименования. Необходимый параметр."
-          locale: "Язык. По умолчанию: base"
-          locale_to_translate_from: "Язык, с которого переводить (по умолчанию: base)"
-          locales_filter: "Список языков для обработки, разделенный запятыми (,). По умолчанию: все.
-            Специальное значение: base."
-          missing_types: "Типы недостающих переводов: %{valid}. По умолчанию: все"
-          new_key_name: "Новое имя, интерполирует оригинальное название как %{key}. Необходимый параметр."
-          nostdin: "Не читать дерево из стандартного ввода"
-          out_format: "Формат вывода: %{valid_text}. %{default_text}."
-          pattern_router: "Использовать pattern_router: ключи распределятся по файлам согласно data.write"
+          confirm: Подтвердить автоматом
+          data_format: 'Формат данных: %{valid_text}. %{default_text}.'
+          key_pattern: Маска ключа (например, common.*)
+          key_pattern_to_rename: Полный ключ (шаблон) для переименования. Необходимый параметр.
+          locale: 'Язык. По умолчанию: base'
+          locale_to_translate_from: 'Язык, с которого переводить (по умолчанию: base)'
+          locales_filter: >-
+            Список языков для обработки, разделенный запятыми (,). По умолчанию: все. Специальное
+            значение: base.
+          missing_types: 'Типы недостающих переводов: %{valid}. По умолчанию: все'
+          new_key_name: Новое имя, интерполирует оригинальное название как %{key}. Необходимый параметр.
+          nostdin: Не читать дерево из стандартного ввода
+          out_format: 'Формат вывода: %{valid_text}. %{default_text}.'
+          pattern_router: 'Использовать pattern_router: ключи распределятся по файлам согласно data.write'
           strict: Не угадывать динамические использования ключей, например `t("category.#{category.key}")`
-          value: "Значение, интерполируется с %{value}, %{human_key}, %{key}, %{default}, %{value_or_human_key}, %{value_or_default_or_human_key}"
+          value: >-
+            Значение, интерполируется с %{value}, %{human_key}, %{key}, %{default}, %{value_or_human_key},
+            %{value_or_default_or_human_key}
       desc:
-        add_missing: "добавить недостающие ключи к переводам"
-        config: "показать конфигурацию"
-        data: "показать данные переводов"
-        data_merge: "добавить дерево к переводам"
-        data_remove: "удалить ключи, которые есть в дереве, из данных"
-        data_write: "заменить переводы деревом"
-        eq_base: "показать переводы, равные значениям в основном языке"
-        find: "показать, где ключи используются в коде"
-        gem_path: "показать путь к ruby gem"
-        health: "Всё ОК?"
-        irb: "начать REPL сессию в контексте i18n-tasks"
-        missing: "показать недостающие переводы"
-        normalize: "нормализовать файлы переводов (сортировка и распределение)"
-        remove_unused: "удалить неиспользуемые ключи"
-        translate_missing: "перевести недостающие переводы с Google Translate"
-        tree_convert: "преобразовать дерево между форматами"
-        tree_filter: "фильтровать дерево по ключу"
-        tree_merge: "объединенить деревья"
-        tree_rename_key: "переименовать узел дерева"
-        tree_set_value: "заменить значения ключей"
-        tree_subtract: "дерево A минус ключи в дереве B"
-        tree_translate: "Перевести дерево при помощи Google Translate на язык корневых узлов"
-        unused: "показать неиспользуемые переводы"
-        xlsx_report: "сохранить недостающие и неиспользуемые переводы в Excel-файл"
+        add_missing: добавить недостающие ключи к переводам
+        config: показать конфигурацию
+        data: показать данные переводов
+        data_merge: добавить дерево к переводам
+        data_remove: удалить ключи, которые есть в дереве, из данных
+        data_write: заменить переводы деревом
+        eq_base: показать переводы, равные значениям в основном языке
+        find: показать, где ключи используются в коде
+        gem_path: показать путь к ruby gem
+        health: Всё ОК?
+        irb: начать REPL сессию в контексте i18n-tasks
+        missing: показать недостающие переводы
+        normalize: нормализовать файлы переводов (сортировка и распределение)
+        remove_unused: удалить неиспользуемые ключи
+        translate_missing: перевести недостающие переводы с Google Translate
+        tree_convert: преобразовать дерево между форматами
+        tree_filter: фильтровать дерево по ключу
+        tree_merge: объединенить деревья
+        tree_rename_key: переименовать узел дерева
+        tree_set_value: заменить значения ключей
+        tree_subtract: дерево A минус ключи в дереве B
+        tree_translate: Перевести дерево при помощи Google Translate на язык корневых узлов
+        unused: показать неиспользуемые переводы
+        xlsx_report: сохранить недостающие и неиспользуемые переводы в Excel-файл
       encourage:
-      - "Хорошая работа!"
-      - "Отлично!"
-      - "Прекрасно!"
+      - Хорошая работа!
+      - Отлично!
+      - Прекрасно!
       enum_list_opt:
         invalid: "%{invalid} не в: %{valid}."
       enum_opt:
         invalid: "%{invalid} не является одним из: %{valid}."
       errors:
-        invalid_format: "Неизвестный формат %{invalid}. Форматы: %{valid}."
-        invalid_locale: "Неверный язык %{invalid}"
+        invalid_format: 'Неизвестный формат %{invalid}. Форматы: %{valid}.'
+        invalid_locale: Неверный язык %{invalid}
         invalid_missing_type:
-          one: "Неизвестный тип %{invalid}. Типы: %{valid}."
-          other: "Неизвестные типы: %{invalid}. Типы: %{valid}."
-        pass_forest: "Передайте дерево"
+          one: 'Неизвестный тип %{invalid}. Типы: %{valid}.'
+          other: 'Неизвестные типы: %{invalid}. Типы: %{valid}.'
+        pass_forest: Передайте дерево
     common:
-      base_value: "Исходное значение"
-      continue_q: "Продолжить?"
-      key: "Ключ"
-      locale: "Язык"
-      n_more: "ещё %{count}"
-      type: "Тип"
-      value: "Значение"
+      base_value: Исходное значение
+      continue_q: Продолжить?
+      key: Ключ
+      locale: Язык
+      n_more: ещё %{count}
+      type: Тип
+      value: Значение
     data_stats:
-      text: "%{key_count} ключей в %{locale_count} языках. В среднем, длина строки: %{value_chars_avg},
-        сегменты ключей: %{key_segments_avg}, ключей в языке %{per_locale_avg}."
-      text_single_locale: "%{key_count} ключей. В среднем, длина строки: %{value_chars_avg}, сегменты
-        ключей: %{key_segments_avg}."
-      title: "Данные (%{locales}):"
+      text: >-
+        %{key_count} ключей в %{locale_count} языках. В среднем, длина строки: %{value_chars_avg},
+        сегменты ключей: %{key_segments_avg}, ключей в языке %{per_locale_avg}.
+      text_single_locale: >-
+        %{key_count} ключей. В среднем, длина строки: %{value_chars_avg}, сегменты ключей: %{key_segments_avg}.
+      title: 'Данные (%{locales}):'
     google_translate:
       errors:
-        no_api_key: "Задайте ключ API Google через переменную окружения GOOGLE_TRANSLATE_API_KEY или
-          translation.api_key в config/i18n-tasks.yml. Получите ключ через https://code.google.com/apis/console."
-        no_results: Google Translate не дал результатов. Убедитесь в том, что платежная информация
-          добавлена в https://code.google.com/apis/console.
+        no_api_key: >-
+          Задайте ключ API Google через переменную окружения GOOGLE_TRANSLATE_API_KEY или translation.api_key
+          в config/i18n-tasks.yml. Получите ключ через https://code.google.com/apis/console.
+        no_results: >-
+          Google Translate не дал результатов. Убедитесь в том, что платежная информация добавлена
+          в https://code.google.com/apis/console.
     health:
-      no_keys_detected: "Ключи не обнаружены. Проверьте data.read в config/i18n-tasks.yml."
+      no_keys_detected: Ключи не обнаружены. Проверьте data.read в config/i18n-tasks.yml.
     missing:
-      details_title: "На других языках или в коде"
-      none: "Всё переведено."
+      details_title: На других языках или в коде
+      none: Всё переведено.
     remove_unused:
       confirm:
-        one: "Один перевод будут удалён из %{locales}."
-        other: "Переводы (%{count}) будут удалены из %{locales}."
-      noop: "Нет неиспользуемых ключей"
-      removed: "Удалены ключи (%{count})"
+        one: Один перевод будут удалён из %{locales}.
+        other: Переводы (%{count}) будут удалены из %{locales}.
+      noop: Нет неиспользуемых ключей
+      removed: Удалены ключи (%{count})
     translate_missing:
-      translated: "Переведены ключи (%{count})"
+      translated: Переведены ключи (%{count})
     unused:
-      none: "Все переводы используются."
+      none: Все переводы используются.
     usages:
-      none: "Не найдено использований."
+      none: Не найдено использований.

data/i18n-tasks.gemspec

@@ -1,14 +1,15 @@
+# frozen_string_literal: true
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'i18n/tasks/version'
 
-Gem::Specification.new do |s|
+Gem::Specification.new do |s| # rubocop:disable Metrics/BlockLength
   s.name          = 'i18n-tasks'
   s.version       = I18n::Tasks::VERSION
   s.authors       = ['glebm']
   s.email         = ['glex.spb@gmail.com']
   s.license       = 'MIT'
-  s.summary       = %q{Manage localization and translation with the awesome power of static analysis}
+  s.summary       = 'Manage localization and translation with the awesome power of static analysis'
   s.description   = <<-TEXT
 i18n-tasks helps you find and manage missing and unused translations.
 
@@ -21,16 +22,14 @@ cp $(i18n-tasks gem-path)/templates/config/i18n-tasks.yml config/
 # Add an RSpec for missing and unused keys:
 cp $(i18n-tasks gem-path)/templates/rspec/i18n_spec.rb spec/
 TEXT
-  s.homepage      = 'https://github.com/glebm/i18n-tasks'
+  s.homepage = 'https://github.com/glebm/i18n-tasks'
   if s.respond_to?(:metadata=)
     s.metadata = { 'issue_tracker' => 'https://github.com/glebm/i18n-tasks' }
   end
-  if s.respond_to?(:required_ruby_version=)
-    s.required_ruby_version = '~> 2.1'
-  end
+  s.required_ruby_version = '~> 2.1' if s.respond_to?(:required_ruby_version=)
 
-  s.files         = `git ls-files`.split($/)
-  s.files         -= s.files.grep(%r{^(doc/|\.|spec/)}) + %w(CHANGES.md config/i18n-tasks.yml Gemfile)
+  s.files = `git ls-files`.split($/)
+  s.files -= s.files.grep(%r{^(doc/|\.|spec/)}) + %w(CHANGES.md config/i18n-tasks.yml Gemfile)
   s.executables   = s.files.grep(%r{^bin/}) { |f| File.basename(f) } - %w(i18n-tasks.cmd)
   s.test_files    = s.files.grep(%r{^(test|spec|features)/})
   s.require_paths = ['lib']
@@ -48,5 +47,7 @@ TEXT
   s.add_development_dependency 'bundler', '~> 1.3'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec', '~> 3.3'
+  s.add_development_dependency 'rubocop'
+  s.add_development_dependency 'simplecov'
   s.add_development_dependency 'yard'
 end

data/lib/i18n/tasks.rb

@@ -11,9 +11,7 @@ module I18n
         @verbose
       end
 
-      def verbose=(value)
-        @verbose = value
-      end
+      attr_writer :verbose
 
       # Add a scanner to the default configuration.
       #
@@ -37,14 +35,13 @@ module I18n
       end
     end
 
-    @verbose = !!ENV['VERBOSE']
+    @verbose = !ENV['VERBOSE'].nil?
 
     module Data
     end
   end
 end
 
-
 require 'active_support/inflector'
 require 'active_support/core_ext/hash'
 require 'active_support/core_ext/array/access'

data/lib/i18n/tasks/cli.rb

@@ -10,16 +10,13 @@ class I18n::Tasks::CLI
     new.start(argv)
   end
 
-  def initialize
-  end
+  def initialize; end
 
   def start(argv)
     I18n.with_locale(base_task.internal_locale) do
       auto_output_coloring do
         begin
-          if run(argv) == :exit_1
-            exit 1
-          end
+          exit 1 if run(argv) == :exit_1
         rescue OptionParser::ParseError => e
           error e.message, 64
         rescue I18n::Tasks::CommandError => e
@@ -51,7 +48,7 @@ class I18n::Tasks::CLI
   def commands
     # load base task to initialize plugins
     base_task
-    @commands ||= ::I18n::Tasks::Commands.cmds.transform_keys { |k| k.to_s.tr('_'.freeze, '-'.freeze) }
+    @commands ||= ::I18n::Tasks::Commands.cmds.transform_keys { |k| k.to_s.tr('_', '-') }
   end
 
   private
@@ -64,7 +61,7 @@ class I18n::Tasks::CLI
     command = parse_command! argv
     options = optparse! command, argv
     parse_options! options, command, argv
-    [command.tr('-'.freeze, '_'.freeze), options.update(arguments: argv)]
+    [command.tr('-', '_'), options.update(arguments: argv)]
   end
 
   def optparse!(command, argv)
@@ -103,14 +100,14 @@ class I18n::Tasks::CLI
 
   def allow_help_arg_first!(argv)
     # allow `i18n-tasks --help command` in addition to `i18n-tasks command --help`
-    if %w(-h --help).include?(argv[0]) && argv[1] && !argv[1].start_with?('-'.freeze)
+    if %w(-h --help).include?(argv[0]) && argv[1] && !argv[1].start_with?('-')
       argv[0], argv[1] = argv[1], argv[0]
     end
   end
 
   def parse_command!(argv)
     allow_help_arg_first! argv
-    if argv[0] && !argv[0].start_with?('-'.freeze)
+    if argv[0] && !argv[0].start_with?('-')
       if commands.keys.include?(argv[0])
         argv.shift
       else
@@ -120,9 +117,9 @@ class I18n::Tasks::CLI
   end
 
   def verbose_option(op)
-    op.on('--verbose', 'Verbose output') {
+    op.on('--verbose', 'Verbose output') do
       ::I18n::Tasks.verbose = true
-    }
+    end
   end
 
   def help_option(op)
@@ -157,14 +154,14 @@ class I18n::Tasks::CLI
   def parse_options!(options, command, argv)
     commands[command][:args].each do |flag|
       name          = option_name flag
-      options[name] = parse_option flag, options[name], argv, self.context
+      options[name] = parse_option flag, options[name], argv, context
     end
   end
 
   def parse_option(flag, val, argv, context)
     conf = flag.last.is_a?(Hash) ? flag.last : {}
     if conf[:consume_positional]
-      val = Array(val) + Array(flag.include?(Array) ? argv.flat_map { |x| x.split(','.freeze) } : argv)
+      val = Array(val) + Array(flag.include?(Array) ? argv.flat_map { |x| x.split(',') } : argv)
     end
     val = conf[:default] if val.nil? && conf.key?(:default)
     val = conf[:parser].call(val, context) if conf.key?(:parser)
@@ -172,9 +169,9 @@ class I18n::Tasks::CLI
   end
 
   def option_name(flag)
-    flag.detect { |f|
-      f.start_with?('--'.freeze)
-    }.sub(/\A--(\[no-\])?/, ''.freeze).sub(/[^\-\w].*\z/, ''.freeze).to_sym
+    flag.detect do |f|
+      f.start_with?('--')
+    end.sub(/\A--(\[no-\])?/, '').sub(/[^\-\w].*\z/, '').to_sym
   end
 
   def try_call(v)
@@ -190,7 +187,7 @@ class I18n::Tasks::CLI
     fail ExecutionError.new(message, exit_code)
   end
 
-  class ExecutionError < Exception
+  class ExecutionError < RuntimeError
     attr_reader :exit_code
 
     def initialize(message, exit_code)
@@ -206,5 +203,4 @@ class I18n::Tasks::CLI
   ensure
     Term::ANSIColor.coloring = coloring_was
   end
-
 end

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

@@ -10,7 +10,6 @@ module I18n::Tasks
 
       attr_reader :i18n
 
-
       # @param [I18n::Tasks::BaseTask] i18n
       def initialize(i18n)
         @i18n = i18n
@@ -18,8 +17,8 @@ module I18n::Tasks
 
       def run(name, opts = {})
         name = name.to_sym
-        public_name = name.to_s.tr '_'.freeze, '-'.freeze
-        log_verbose "task: #{public_name}(#{opts.map { |k, v| "#{k}: #{v.inspect}" } * ', '.freeze})"
+        public_name = name.to_s.tr '_', '-'
+        log_verbose "task: #{public_name}(#{opts.map { |k, v| "#{k}: #{v.inspect}" } * ', '})"
         if opts.empty? || method(name).arity.zero?
           send name
         else