Fixes rubocop offenses

glebm · May 7, 2024 · 9ca08b0 · 9ca08b0
1 parent 88de40e
commit 9ca08b0
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 15 deletions.
diff --git a/lib/i18n/tasks/key_pattern_matching.rb b/lib/i18n/tasks/key_pattern_matching.rb
@@ -31,10 +31,10 @@ def compile_key_pattern(key_pattern)
 
   def key_pattern_re_body(key_pattern)
     key_pattern
-      .gsub(/\./, '\.')
-      .gsub(/\*:/, '[^.]+?')
-      .gsub(/\*/, '.*')
-      .gsub(/:/, '(?<=^|\.)[^.]+?(?=\.|$)')
+      .gsub('.', '\.')
+      .gsub('*:', '[^.]+?')
+      .gsub('*', '.*')
+      .gsub(':', '(?<=^|\.)[^.]+?(?=\.|$)')
       .gsub(/\{(.*?)}/) { "(#{Regexp.last_match(1).strip.gsub(/\s*,\s*/, '|')})" }
   end
 end
diff --git a/lib/i18n/tasks/scanners/relative_keys.rb b/lib/i18n/tasks/scanners/relative_keys.rb
@@ -61,7 +61,7 @@ def prefix(normalized_path, calling_method: nil)
             "#{file_key.sub(/_controller$/, '')}.#{calling_method}"
           else
             # Remove _ prefix from partials
-            file_key.gsub(/\._/, DOT)
+            file_key.gsub('._', DOT)
           end
         end
       end

diff --git a/lib/i18n/tasks/scanners/ruby_ast_scanner.rb b/lib/i18n/tasks/scanners/ruby_ast_scanner.rb
@@ -81,7 +81,7 @@ def comments_to_occurences(path, ast, comments)
         results = []
 
         # method_name is not available at this stage
-        calls.each do |send_node, _method_name|
+        calls.each do |(send_node, _method_name)|
           @matchers.each do |matcher|
             result = matcher.convert_to_key_occurrences(
               send_node,

diff --git a/lib/i18n/tasks/translators/google_translator.rb b/lib/i18n/tasks/translators/google_translator.rb
@@ -23,7 +23,7 @@ def options_for_translate_values(from:, to:, **options)
       options.merge(
         api_key: api_key,
         from: from,
-        to: to,
+        to: to
       )
     end
 

diff --git a/spec/google_translate_spec.rb b/spec/google_translate_spec.rb
@@ -80,7 +80,7 @@
         'config/locales/zh-tw.yml' => '',
         'config/locales/zh-hant.yml' => '',
         'config/locales/zh-hk.yml' => '',
-        'config/locales/es.yml' => '',
+        'config/locales/es.yml' => ''
       )
     end
 
@@ -99,18 +99,18 @@
           task.data[:en] = build_tree('en' => { 'common' => { 'a' => 'λ', 'horse' => 'horse' } })
 
           # Loading translations seems to require at least one existing value.
-          %w(zh zh-cn zh-hans zh-tw zh-hant zh-hk).each do |locale|
+          %w[zh zh-cn zh-hans zh-tw zh-hant zh-hk].each do |locale|
             task.data[locale] = build_tree(locale => { 'common' => { 'a' => 'λ' } })
           end
 
           run_cmd 'translate-missing'
 
-          expect(task.t('common.horse', 'zh'     )).to eq("马") # Simplified Chinese
-          expect(task.t('common.horse', 'zh-cn'  )).to eq("马") # Simplified Chinese
-          expect(task.t('common.horse', 'zh-hans')).to eq("马") # Simplified Chinese
-          expect(task.t('common.horse', 'zh-tw'  )).to eq("馬") # Traditional Chinese
-          expect(task.t('common.horse', 'zh-hant')).to eq("馬") # Traditional Chinese
-          expect(task.t('common.horse', 'zh-hk'  )).to eq("馬") # Traditional Chinese
+          expect(task.t('common.horse', 'zh')).to eq('马') # Simplified Chinese
+          expect(task.t('common.horse', 'zh-cn')).to eq('马') # Simplified Chinese
+          expect(task.t('common.horse', 'zh-hans')).to eq('马') # Simplified Chinese
+          expect(task.t('common.horse', 'zh-tw')).to eq('馬') # Traditional Chinese
+          expect(task.t('common.horse', 'zh-hant')).to eq('馬') # Traditional Chinese
+          expect(task.t('common.horse', 'zh-hk')).to eq('馬') # Traditional Chinese
         end
       end
     end