Adding Super Linter (#7864)

magento · Sep 14, 2020 · 2359c12 · 2359c12
1 parent 05bebb6
commit 2359c12
Show file tree

Hide file tree

Showing 109 changed files with 465 additions and 486 deletions.
diff --git a/.github/linters/.markdownlint.json b/.github/linters/.markdownlint.json
@@ -0,0 +1,25 @@
+{
+  "default": false,
+  "MD001": true,
+  "MD003": { "style": "atx" },
+  "MD004": true,
+  "MD005": true,
+  "MD006": true,
+  "MD007": { "indent": 3 },
+  "MD009": true,
+  "MD010": true,
+  "MD012": true,
+  "MD018": true,
+  "MD019": true,
+  "MD023": true,
+  "MD025": true,
+  "MD029": true,
+  "MD030": { "ol_single": 1, "ol_multi": 1, "ul_single": 2, "ul_multi": 2 },
+  "MD032": true,
+  "MD035": true,
+  "MD036": true,
+  "MD037": true,
+  "MD038": true,
+  "MD040": true,
+  "MD046": { "style": "fenced"}
+}
diff --git a/.github/linters/.ruby-lint.yml b/.github/linters/.ruby-lint.yml
@@ -0,0 +1,11 @@
+---
+#######################
+# Rubocop Config file #
+#######################
+
+inherit_gem:
+  rubocop-github:
+    - config/default.yml
+
+Style/StringLiterals:
+  EnforcedStyle: single_quotes
diff --git a/.github/workflows/linter.yml b/.github/workflows/linter.yml
@@ -0,0 +1,53 @@
+---
+###########################
+###########################
+## Linter GitHub Actions ##
+###########################
+###########################
+name: Lint Code Base
+
+#
+# Documentation:
+# https://help.github.com/en/articles/workflow-syntax-for-github-actions
+#
+
+#############################
+# Start the job on all push #
+#############################
+on:
+  pull_request:
+    branches:
+      - master
+      - 2.*-develop
+
+###############
+# Set the Job #
+###############
+jobs:
+
+  build:
+    # Name the Job
+    name: Lint Code Base
+    # Set the agent to run on
+    runs-on: ubuntu-latest
+
+    ##################
+    # Load all steps #
+    ##################
+    steps:
+      ##########################
+      # Checkout the code base #
+      ##########################
+      - name: Checkout Code
+        uses: actions/checkout@v2
+
+      ################################
+      # Run Linter against code base #
+      ################################
+      - name: Lint Code Base
+        uses: github/super-linter@v3
+        env:
+          VALIDATE_ALL_CODEBASE: false
+          DEFAULT_BRANCH: master
+          VALIDATE_HTML: false
+          VALIDATE_MD: false
diff --git a/.markdownlint.json b/.markdownlint.json
@@ -0,0 +1 @@
+.github/linters/.markdownlint.json
diff --git a/Gemfile b/Gemfile
@@ -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
diff --git a/Rakefile b/Rakefile
@@ -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
diff --git a/_plugins/algolia.rb b/_plugins/algolia.rb
@@ -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

diff --git a/_plugins/liquid-tags/collapsible.rb b/_plugins/liquid-tags/collapsible.rb
@@ -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

diff --git a/_plugins/page-params/github-path.rb → _plugins/page-params/github_path.rb b/_plugins/page-params/github-path.rb → _plugins/page-params/github_path.rb
@@ -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

diff --git a/_plugins/page-params/last-modified-at.rb → _plugins/page-params/last_modified_at.rb b/_plugins/page-params/last-modified-at.rb → _plugins/page-params/last_modified_at.rb
@@ -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)
 

diff --git a/...ins/page-params/page-baseurl-generator.rb → ...ins/page-params/page_baseurl_generator.rb b/...ins/page-params/page-baseurl-generator.rb → ...ins/page-params/page_baseurl_generator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 

diff --git a/_plugins/page-params/page-canonical-url.rb → _plugins/page-params/page_canonical_url.rb b/_plugins/page-params/page-canonical-url.rb → _plugins/page-params/page_canonical_url.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright © Magento, Inc. All rights reserved.
 # See COPYING.txt for license details.
 

diff --git a/_plugins/page-params/page-versionless.rb → _plugins/page-params/page_versionless.rb b/_plugins/page-params/page-versionless.rb → _plugins/page-params/page_versionless.rb
@@ -17,6 +17,6 @@
   filtering_pattern = '/guides/v'
 
   next if page.url.start_with? filtering_pattern
-  
+
   page.data['versionless'] = true
-end
+end
diff --git a/_plugins/page-params/page-versions.rb → _plugins/page-params/page_versions.rb b/_plugins/page-params/page-versions.rb → _plugins/page-params/page_versions.rb
@@ -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)
 

diff --git a/bin/htmlproofer b/bin/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("html-proofer", "htmlproofer")
+load Gem.bin_path('html-proofer', 'htmlproofer')
diff --git a/bin/image_optim b/bin/image_optim
@@ -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')
diff --git a/bin/jekyll b/bin/jekyll
@@ -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'

diff --git a/bin/kramdown b/bin/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("kramdown", "kramdown")
+load Gem.bin_path('kramdown', 'kramdown')
diff --git a/bin/mdl b/bin/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("mdl", "mdl")
+load Gem.bin_path('mdl', 'mdl')