Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add jsx-a11y to linting #288

Closed
landonreed opened this issue Sep 11, 2019 · 2 comments · Fixed by #289
Closed

Add jsx-a11y to linting #288

landonreed opened this issue Sep 11, 2019 · 2 comments · Fixed by #289
Assignees
@evansiroky
Labels
enhancement released

Comments

@landonreed
Copy link
Contributor

@evansiroky, there is a requirement for a potential trip planning customer to adhere to WCAG 2.1. Given this, I think it would be helpful to add jsx-a11y to the lint rules, or at least somehow enable this for certain projects like otp-rr.

I did see this brought up in this issue. Do you have any objections to adding this to the linter?
@evansiroky
Copy link
Contributor

No objections.

landonreed added a commit that referenced this issue Sep 16, 2019
@landonreed
feat(a11y): re-enable jsx-a11y for linting
58df413 
fix #288
@landonreed landonreed mentioned this issue Sep 16, 2019
Merged
@trevorgerhardt
Copy link
Member

🎉 This issue has been resolved in version 5.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@evansiroky evansiroky

Labels
enhancement released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Re-enable jsx-a11y for linting conveyal/mastarm
3 participants
@trevorgerhardt @landonreed @evansiroky