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.

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

add eslint-plugin-jsx-a11y #3894

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bjornalm
Copy link
Collaborator

What changes were proposed in this pull request?

Improve accessibility by adding eslint-plugin-jsx-a11y to the linter

How was this patch tested?

  • (Please explain how this patch was tested. Ex: unit tests, manual tests)
  • (If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

Please review Hue Contributing Guide before opening a pull request.

Copy link

⚠️ No unit test files modified. Please ensure that changes are properly tested. ⚠️

1 similar comment
Copy link

⚠️ No unit test files modified. Please ensure that changes are properly tested. ⚠️

@bjornalm bjornalm force-pushed the bjorn--add-accessibility-linting branch from e236359 to 96554b0 Compare January 7, 2025 11:03
Copy link

github-actions bot commented Jan 7, 2025

⚠️ No unit test files modified. Please ensure that changes are properly tested. ⚠️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant