Skip to content

Conversation

@khiga8
Copy link
Collaborator

@khiga8 khiga8 commented Feb 22, 2022

Depends on #19. e953e0b should be removed once the PR 19 has merged..

Closes #20

This PR converts NoAriaLabelMisuse to a counter so that when people introduce violations to a file, they need to bump the count. In dotcom, this rule is the most common linter violation and we've needed to add a bunch of file-level disables. Let's convert this to a counter so this rule isn't ignored.

@khiga8 khiga8 marked this pull request as ready for review February 22, 2022 20:57
@khiga8 khiga8 requested review from a team, bolonio and smockle February 22, 2022 20:58
Copy link

@bolonio bolonio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As in the other PR: From not being an erb expert, it looks good to me, if the tests have been passed :)

@khiga8 khiga8 force-pushed the kh-convert-aria-label-misuse-to-counter branch from 3c67a14 to 83f6534 Compare February 23, 2022 18:06
@khiga8 khiga8 merged commit 19b3936 into main Feb 23, 2022
@khiga8 khiga8 deleted the kh-convert-aria-label-misuse-to-counter branch February 23, 2022 18:09
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.

Convert NoAriaLabelMisuse to a counter

3 participants