Skip to content

Commit

Permalink
Remove unnecessary eslint ignores
Browse files Browse the repository at this point in the history
The bug with the `control-has-associated-label` rule was fixed in
`eslint-plugin-jsx-a11y@6.10.0` so the ignore comments are no
longer required.
  • Loading branch information
AlanGreene authored and tekton-robot committed Sep 13, 2024
1 parent 6c6b64e commit de33c7f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,6 @@ function HelpIcon({ title }) {

return (
<Tooltip align="top-end" autoAlign label={title}>
{/* false positive, Carbon automatically adds aria-labelledby to the button */}
{/* eslint-disable-next-line jsx-a11y/control-has-associated-label */}
<button className="tkn--tooltip-trigger" type="button">
<Information />
</button>
Expand Down
2 changes: 0 additions & 2 deletions src/containers/ImportResources/ImportResources.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ const initialMethod = 'apply';

const HelpIcon = ({ title }) => (
<Tooltip align="top-left" label={title}>
{/* false positive, Carbon automatically adds aria-labelledby to the button */}
{/* eslint-disable-next-line jsx-a11y/control-has-associated-label */}
<button className="tkn--tooltip-trigger" type="button">
<Information />
</button>
Expand Down

0 comments on commit de33c7f

Please sign in to comment.