Cleanup Ruby (with RuboCop)

Author: MikeMcQuaid

Rakefile

@@ -6,4 +6,4 @@ Rake::TestTask.new do |t|
   t.verbose = false
 end
 
-task :default => :test
+task default: :test

script/html-proofer

@@ -5,15 +5,16 @@ require "html-proofer"
 
 url_ignores = [
   %r{^https://github.com/github/opensource.guide},
-  'https://karissa.github.io/post/okf-de',
-  'https://this-week-in-rust.org/',
-  'https://www.vagrantup.com/',
-  'https://www.quora.com/'
+  "https://karissa.github.io/post/okf-de",
+  "https://this-week-in-rust.org/",
+  "https://www.vagrantup.com/",
+  "https://www.quora.com/",
 ]
 
-HTMLProofer::Runner.new(["_site"],
-  :type => :directory,
-  :url_ignore => url_ignores,
-  :verbose => !!ENV["VERBOSE"],
-  :check_html => true
+HTMLProofer::Runner.new(
+  ["_site"],
+  type: :directory,
+  url_ignore: url_ignores,
+  verbose: !!ENV["VERBOSE"],
+  check_html: true
 ).run

test/helper.rb

@@ -8,7 +8,7 @@ class << self
 end
 
 def source
-  File.expand_path('../', File.dirname(__FILE__))
+  File.expand_path("../", File.dirname(__FILE__))
 end
 
 def config
@@ -20,7 +20,7 @@ def docs
 end
 
 def pages
-  docs.map { |doc| doc.to_liquid }
+  docs.map(&:to_liquid)
 end
 
 def site

test/lint_test.rb

@@ -1,14 +1,11 @@
 require_relative "./helper"
 
 describe "lint test" do
-
   pages.each do |page|
-    next unless page["path"].match(/\.md$/)
+    next unless page["path"].match?(/\.md$/)
 
     describe page["path"] do
-
       describe "frontmatter" do
-
         before do
           # Load raw metadata to skip defaults
           @data = SafeYAML.load_file(page["path"])
@@ -26,19 +23,16 @@ def assert_valid_fields(data, fields)
     assert extra_fields.empty?, "Unexpected metadata: #{extra_fields.inspect}"
 
     fields.each do |name, attrs|
-      if attrs["required"]
-        assert data.key?(name), "#{name} is required"
-      end
+      assert data.key?(name), "#{name} is required" if attrs["required"]
 
       if attrs["type"] && @data[name]
         assert_kind_of Kernel.const_get(attrs["type"]), @data[name]
       end
 
       # Check subfields
-      if attrs["fields"] && @data[name]
-        @data[name].each do |d|
-          assert_valid_fields(d, attrs["fields"])
-        end
+      next unless attrs["fields"] && @data[name]
+      @data[name].each do |d|
+        assert_valid_fields(d, attrs["fields"])
       end
     end
   end