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

chore(eslint-config): change eslint to better match strict ruleset #24920

Merged
merged 2 commits into from
Jun 18, 2020

Conversation

herecydev
Copy link
Contributor

@herecydev herecydev commented Jun 11, 2020

Refine a11y linting rules so that some trigger alerts or dialogs instead of warnings.

Fixes #24582

@herecydev herecydev requested a review from madalynrose June 11, 2020 11:37
@herecydev herecydev requested a review from a team as a code owner June 11, 2020 11:37
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jun 11, 2020
@danabrit danabrit changed the title change eslint to better match strict ruleset chore: change eslint to better match strict ruleset Jun 11, 2020
@danabrit danabrit changed the title chore: change eslint to better match strict ruleset chore(eslint-config): change eslint to better match strict ruleset Jun 11, 2020
@danabrit
Copy link
Contributor

@herecydev Thank you for your pull request! I updated your PR's description and title to conform with Conventional Commit guidelines. Please confirm that the description is accurate, and correct it if not. 💜

@danabrit danabrit added status: awaiting author response Additional information has been requested from the author type: chore and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jun 11, 2020
@madalynrose madalynrose removed the status: awaiting author response Additional information has been requested from the author label Jun 18, 2020
@madalynrose
Copy link
Contributor

Hey there @herecydev! Apologies for the delay in response! Thank you so much for making this!

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.

Better eslint a11y rule defaults
3 participants