Autofix Ruby code with rubocop

magento · dshevtsov · Sep 14, 2020 · Sep 11, 2020 · Sep 11, 2020 · Sep 11, 2020
commit b89e9cbac7fa1fd9210655465f38230e66cd7977
@@ -4,15 +4,15 @@ source 'https://rubygems.org'
 
 gem 'devdocs', git: 'https://github.com/magento-devdocs/devdocs-theme.git'
 gem 'jekyll'
-gem 'whatsup_github'
 gem 'rake'
+gem 'whatsup_github'
 
 gem 'wdm', platform: :mswin
 
 group :test do
   gem 'html-proofer'
-  gem 'mdl'
   gem 'launchy'
+  gem 'mdl'
 end
 
 group :optimization do
@@ -22,9 +22,9 @@ end
 
 group :jekyll_plugins do
   gem 'jekyll-algolia', '~> 1.0'
+  gem 'jekyll-optional-front-matter'
   gem 'jekyll-redirect-from'
+  gem 'jekyll-relative-links'
   gem 'jekyll-sitemap'
   gem 'jekyll-titles-from-headings'
-  gem 'jekyll-relative-links'
-  gem 'jekyll-optional-front-matter'
 end
@@ -71,7 +71,7 @@ desc 'Generate data for a news digest. Default timeframe is a week since today.
 task :whatsnew do
   date = ENV['since']
   print 'Generating data for the weekly digest: $ '.magenta
-  if date.nil? or date.empty?
+  if date.nil? || date.empty?
     sh 'bin/whatsup_github'
   elsif date.is_a? String
     sh 'bin/whatsup_github', 'since', ENV['since'].to_s
@@ -84,8 +84,8 @@ desc 'Generate index for Algolia'
 task index: %w[init] do
   puts 'Generating index for Algolia ...'
   sh 'bin/jekyll',
-        'algolia',
-          '--config=_config.yml,_config.index.yml'
+     'algolia',
+     '--config=_config.yml,_config.index.yml'
 end
 
 desc 'Convert HTML text to kramdown in your terminal'
@@ -94,4 +94,4 @@ task :convert do
   result = `bin/kramdown --input=html --output=kramdown`
   puts 'Converted text:'.magenta
   puts result.bold
-end
+end
@@ -1,13 +1,15 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 
 module Jekyll
   module Algolia
     module Hooks
-      def self.before_indexing_each(record, node, context)
+      def self.before_indexing_each(record, _node, _context)
         # Do not index records larger than 20000 bytes
-        return nil if record.to_s.bytesize > 20000
-        
+        return nil if record.to_s.bytesize > 20_000
+
         record
       end
     end

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 
@@ -30,7 +32,7 @@ def render(context)
     <b class="collapsible-title">#{@title}</b>
     <div class="collapsible-content">#{content}</div>
   </div>
-        HTML
+      HTML
     end
   end
 
@@ -58,7 +60,7 @@ def render(context)
     <h2 class="collapsible-title">#{@title}</h2>
     <div class="collapsible-content">#{content}</div>
   </div>
-        HTML
+      HTML
     end
   end
 
@@ -85,7 +87,7 @@ def render(context)
     <h3 class="collapsible-title">#{@title}</h3>
     <div class="collapsible-content">#{content}</div>
   </div>
-        HTML
+      HTML
     end
   end
 end

@@ -10,9 +10,8 @@
 # The parameter contains a file path relative to its repository.
 #
 Jekyll::Hooks.register :pages, :post_init do |page|
-
   # Skip virtual pages like MRG topics
-  next if page.kind_of? Jekyll::PageWithoutAFile
+  next if page.is_a? Jekyll::PageWithoutAFile
   # Process only files with 'md' and 'html' extensions
   next unless File.extname(page.path).match?(/md|html/)
   # Skip redirects
@@ -22,7 +21,7 @@
     page.site.source,
     File.dirname(page.path)
   )
-  
+
   filename = File.basename page.path
 
   # Change to the parent directory of the page and read full file path

@@ -25,11 +25,11 @@
   next if page.data['last_modified_at']
 
   # Skip pages created by custom generators like 'mrg_pages'
-  next if page.kind_of? Jekyll::PageWithoutAFile
+  next if page.is_a? Jekyll::PageWithoutAFile
 
   # Add site.source to the page path
   file_path = File.join(page.site.source, page.path)
-  
+
   # Get real path of the page. If this is a symlink read it to get path of the real file with content.
   real_filepath = File.realpath(file_path)
 

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 

@@ -17,6 +17,6 @@
   filtering_pattern = '/guides/v'
 
   next if page.url.start_with? filtering_pattern
-  
+
   page.data['versionless'] = true
-end
+end
@@ -29,6 +29,7 @@
   urls_filtered_by_pattern =
     pages.select do |site_page|
       next if site_page.name == 'redirect.html'
+
       site_page.url.start_with? filtering_pattern
     end.map(&:url)
 

@@ -8,11 +8,11 @@
 # this file is here to facilitate running it.
 #
 
-require "pathname"
-ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+require 'pathname'
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
+                                           Pathname.new(__FILE__).realpath)
 
-bundle_binstub = File.expand_path("../bundle", __FILE__)
+bundle_binstub = File.expand_path('bundle', __dir__)
 
 if File.file?(bundle_binstub)
   if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
@@ -23,7 +23,7 @@ Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this
   end
 end
 
-require "rubygems"
-require "bundler/setup"
+require 'rubygems'
+require 'bundler/setup'
 
-load Gem.bin_path("html-proofer", "htmlproofer")
+load Gem.bin_path('html-proofer', 'htmlproofer')
@@ -8,11 +8,11 @@
 # this file is here to facilitate running it.
 #
 
-require "pathname"
-ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+require 'pathname'
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
+                                           Pathname.new(__FILE__).realpath)
 
-bundle_binstub = File.expand_path("../bundle", __FILE__)
+bundle_binstub = File.expand_path('bundle', __dir__)
 
 if File.file?(bundle_binstub)
   if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
@@ -23,7 +23,7 @@ Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this
   end
 end
 
-require "rubygems"
-require "bundler/setup"
+require 'rubygems'
+require 'bundler/setup'
 
-load Gem.bin_path("image_optim", "image_optim")
+load Gem.bin_path('image_optim', 'image_optim')
@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 #
 # This file was generated by Bundler.
 #
@@ -7,8 +9,8 @@
 #
 
 require 'pathname'
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
+                                           Pathname.new(__FILE__).realpath)
 
 require 'rubygems'
 require 'bundler/setup'

@@ -8,11 +8,11 @@
 # this file is here to facilitate running it.
 #
 
-require "pathname"
-ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+require 'pathname'
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
+                                           Pathname.new(__FILE__).realpath)
 
-bundle_binstub = File.expand_path("../bundle", __FILE__)
+bundle_binstub = File.expand_path('bundle', __dir__)
 
 if File.file?(bundle_binstub)
   if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
@@ -23,7 +23,7 @@ Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this
   end
 end
 
-require "rubygems"
-require "bundler/setup"
+require 'rubygems'
+require 'bundler/setup'
 
-load Gem.bin_path("kramdown", "kramdown")
+load Gem.bin_path('kramdown', 'kramdown')
@@ -8,11 +8,11 @@
 # this file is here to facilitate running it.
 #
 
-require "pathname"
-ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+require 'pathname'
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
+                                           Pathname.new(__FILE__).realpath)
 
-bundle_binstub = File.expand_path("../bundle", __FILE__)
+bundle_binstub = File.expand_path('bundle', __dir__)
 
 if File.file?(bundle_binstub)
   if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
@@ -23,7 +23,7 @@ Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this
   end
 end
 
-require "rubygems"
-require "bundler/setup"
+require 'rubygems'
+require 'bundler/setup'
 
-load Gem.bin_path("mdl", "mdl")
+load Gem.bin_path('mdl', 'mdl')
@@ -8,11 +8,11 @@
 # this file is here to facilitate running it.
 #
 
-require "pathname"
-ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+require 'pathname'
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
+                                           Pathname.new(__FILE__).realpath)
 
-bundle_binstub = File.expand_path("../bundle", __FILE__)
+bundle_binstub = File.expand_path('bundle', __dir__)
 
 if File.file?(bundle_binstub)
   if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
@@ -23,7 +23,7 @@ Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this
   end
 end
 
-require "rubygems"
-require "bundler/setup"
+require 'rubygems'
+require 'bundler/setup'
 
-load Gem.bin_path("whatsup_github", "whatsup_github")
+load Gem.bin_path('whatsup_github', 'whatsup_github')
@@ -17,9 +17,7 @@ def run
         @link = create_element(node)
         line = node.line
 
-        if slash?
-          return add_issue('Remove double forward slashes from URLs', line: line)
-        end
+        return add_issue('Remove double forward slashes from URLs', line: line) if slash?
       end
     end
   end

@@ -1,28 +1,29 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 
 namespace :build do
-
   desc 'Build the site for production'
   task prod: %w[init] do
     print 'Building the site for production: $ '.magenta
     sh 'bin/jekyll',
-          'build',
-            '--config=_config.yml,_config.prod.yml',
-            '--verbose',
-            '--profile',
-            '--trace'
+       'build',
+       '--config=_config.yml,_config.prod.yml',
+       '--verbose',
+       '--profile',
+       '--trace'
   end
 
   desc 'Build the site for staging'
   task :stage do
     print 'Building the site for staging: $ '.magenta
     sh 'bin/jekyll',
-          'build',
-            "--baseurl=/#{ENV['BUILD_NUMBER']}",
-            '--config=_config.yml,_config.stage.yml',
-            '--verbose',
-            '--trace',
-            '--profile'
+       'build',
+       "--baseurl=/#{ENV['BUILD_NUMBER']}",
+       '--config=_config.yml,_config.stage.yml',
+       '--verbose',
+       '--trace',
+       '--profile'
   end
 end