73c83303b7 | ||
---|---|---|
.github/workflows | ||
docs | ||
lib | ||
test | ||
.gitignore | ||
.rubocop.yml | ||
CODEOWNERS | ||
Gemfile | ||
Gemfile.lock | ||
LICENSE | ||
README.md | ||
Rakefile | ||
erblint-github.gemspec |
README.md
erblint-github
Template style checking for GitHub's Ruby projects
Setup
- Update your
Gemfile
and runbundle install
gem "erb_lint", require: false
gem "erblint-github"
- Require the linters within the
.erb-linters
folder. This could be done by adding a file.erb-linters/erblint-github.rb
with the following line.
require "erblint-github/linters"
- Update the
erb-lint.yml
to configure the rule.
.erb-lint.yml
---
linters:
GitHub::Accessibility::AriaLabelIsWellFormatted:
enabled: true
GitHub::Accessibility::AvoidBothDisabledAndAriaDisabled:
enabled: true
GitHub::Accessibility::AvoidGenericLinkText:
enabled: true
GitHub::Accessibility::DisabledAttribute:
enabled: true
GitHub::Accessibility::IframeHasTitle:
enabled: true
GitHub::Accessibility::ImageHasAlt:
enabled: true
GitHub::Accessibility::NavigationHasLabel:
enabled: true
GitHub::Accessibility::LinkHasHref:
enabled: true
GitHub::Accessibility::NestedInteractiveElements:
enabled: true
GitHub::Accessibility::NoAriaHiddenOnFocusable:
enabled: true
GitHub::Accessibility::NoAriaLabelMisuse:
enabled: true
GitHub::Accessibility::NoPositiveTabIndex:
enabled: true
GitHub::Accessibility::NoRedundantImageAlt:
enabled: true
GitHub::Accessibility::NoTitleAttribute:
enabled: true
GitHub::Accessibility::SvgHasAccessibleText:
enabled: true
Rules
- GitHub::Accessibility::AriaLabelIsWellFormatted
- GitHub::Accessibility::AvoidBothDisabledAndAriaDisabled
- GitHub::Accessibility::AvoidGenericLinkText
- GitHub::Accessibility::DisabledAttribute
- GitHub::Accessibility::NavigationHasLabel
- GitHub::Accessibility::LinkHasHref
- GitHub::Accessibility::NestedInteractiveElements
- GitHub::Accessibility::IframeHasTitle
- GitHub::Accessibility::ImageHasAlt
- GitHub::Accessibility::NoAriaHiddenOnFocusable
- GitHub::Accessibility::NoAriaLabelMisuse
- GitHub::Accessibility::NoPositiveTabIndex
- GitHub::Accessibility::NoRedundantImageAlt
- GitHub::Accessibility::NoTitleAttribute
- GitHub::Accessibility::SvgHasAccessibleText
Disabling a rule (Deprecated)
This is a soon-to-be deprecated feature. Do not use. See migration guide
erblint
does not natively support rule disables. At GitHub, we've implemented these rules in a way to allow rules to be disabled at an offense-level via counters or disabled at a file-level because often times, we want to enable a rule but aren't able to address all offenses at once. We achieve this in one of two ways.
Rules that are marked as Counter
can be disabled by adding a comment with the offense count that matches the number of offenses within the file like:
<%# erblint:counter GitHub::Accessibility::LinkHasHrefCounter 1 %>
In this comment example, when a new LinkHasHref
offense has been added, the counter will need to be bumped up to 2. More recent rules use a Counter
format.
If you are enabling a rule for the first time and your codebase has a lot of offenses, you can use the -a
command to automatically add these counter comments in the appropriate places.
bundle exec erblint app/views app/components -a
Rules that are not marked as Counter
like NoRedundantImageAlt
are considered to be legacy format. We are in the process of migrating these to counters. These rules can still be disabled at the file-level by adding this comment at the top of the file:
<%# erblint:disable GitHub::Accessibility::NoRedundantImageAlt %>
However, unlike a counter, any subsequent offenses introduced to the file will not raise.
Testing
bundle install
bundle exec rake
Recommended extension
If you use VS Code, we highly encourage ERB Linter extension to see immediate feedback in your editor.
Note
This repo contains several accessibility-related linting rules to help surface accessibility issues that would otherwise go undetected until a later stage. Please note that due to the limitations of static code analysis, these ERB accessibility checks are NOT enough for ensuring the accessibility of your app. This shouldn't be the only tool you use to catch accessibility issues and should be supplemented with other tools that can check the runtime browser DOM output, as well as processes like accessibility design reviews, manual audits, user testing, etc.