Remove GitHub prefix on ERB lint rules

README.md

@@ -23,16 +23,16 @@ require "erblint-github/linters"
 ```yaml 
 ---
 linters:
-  GitHub::Accessibility::ImageHasAlt:
+  Accessibility::ImageHasAlt:
     enabled: true
-  GitHub::Accessibility::NoRedundantImageAlt:
+  Accessibility::NoRedundantImageAlt:
     enabled: true
 ```
 
 ### Rules
 
-- [GitHub::Accessibility::NoRedundantImageAlt](./docs/rules/accessibility/no-redundant-image-alt.md)
-- [GitHub::Accessibility::ImageHasAlt](./docs/rules/accessibility/image-has-alt.md)
+- [Accessibility::NoRedundantImageAlt](./docs/rules/accessibility/no-redundant-image-alt.md)
+- [Accessibility::ImageHasAlt](./docs/rules/accessibility/image-has-alt.md)
 
 ## Testing
 

lib/erblint-github/linters.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-Dir[File.join(__dir__, "linters", "github/**/*.rb")].sort.each { |file| require file }
+Dir[File.join(__dir__, "linters", "**/*.rb")].sort.each { |file| require file }

lib/erblint-github/linters/accessibility/image_has_alt.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require_relative "../custom_helpers"
+
+module ERBLint
+  module Linters
+    module Accessibility
+      class ImageHasAlt < Linter
+        include ERBLint::Linters::CustomHelpers
+        include LinterRegistry
+
+        MESSAGE = "<img> should have an alt prop with meaningful text or an empty string for decorative images"
+
+        def run(processed_source)
+          tags(processed_source).each do |tag|
+            next if tag.name != "img"
+            next if tag.closing?
+
+            alt = possible_attribute_values(tag, "alt")
+
+            generate_offense(self.class, processed_source, tag) if alt.empty?
+          end
+
+          rule_disabled?(processed_source)
+        end
+      end
+    end
+  end
+end

lib/erblint-github/linters/accessibility/no_redundant_image_alt.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require_relative "../custom_helpers"
+
+module ERBLint
+  module Linters
+    module Accessibility
+      class NoRedundantImageAlt < Linter
+        include ERBLint::Linters::CustomHelpers
+        include LinterRegistry
+
+        MESSAGE = "<img> alt prop should not contain `image` or `picture` as screen readers already announce the element as an image"
+        REDUNDANT_ALT_WORDS = %w[image picture].freeze
+
+        def run(processed_source)
+          tags(processed_source).each do |tag|
+            next if tag.name != "img"
+            next if tag.closing?
+
+            alt = possible_attribute_values(tag, "alt").join
+            next if alt.empty?
+
+            generate_offense(self.class, processed_source, tag) if (alt.downcase.split & REDUNDANT_ALT_WORDS).any?
+          end
+
+          rule_disabled?(processed_source)
+        end
+      end
+    end
+  end
+end

test/linters/accessibility/image_has_alt_test.rb

@@ -4,7 +4,7 @@
 
 class ImageHasAltTest < LinterTestCase
   def linter_class
-    ERBLint::Linters::GitHub::Accessibility::ImageHasAlt
+    ERBLint::Linters::Accessibility::ImageHasAlt
   end
 
   def test_warns_if_image_has_no_alt_attribute

test/linters/accessibility/no_redundant_image_alt_test.rb

@@ -4,7 +4,7 @@
 
 class NoRedundantImageAltTest < LinterTestCase
   def linter_class
-    ERBLint::Linters::GitHub::Accessibility::NoRedundantImageAlt
+    ERBLint::Linters::Accessibility::NoRedundantImageAlt
   end
 
   def test_warns_if_alt_contains_image