code review feedback

Author: broccolinisoup

README.md

@@ -32,3 +32,4 @@ ESLint rules for Primer React
 - [direct-slot-children](https://github.com/primer/eslint-plugin-primer-react/blob/main/docs/rules/direct-slot-children.md)
 - [no-deprecated-colors](https://github.com/primer/eslint-plugin-primer-react/blob/main/docs/rules/no-deprecated-colors.md)
 - [no-system-props](https://github.com/primer/eslint-plugin-primer-react/blob/main/docs/rules/no-system-props.md)
+- [a11y-tooltip-non-interactive-trigger](https://github.com/primer/eslint-plugin-primer-react/blob/main/docs/rules/a11y-tooltip-non-interactive-trigger.md)

docs/rules/a11y-tooltip-non-interactive-trigger.md

@@ -1,5 +1,3 @@
-<!-- Write a readme for the non interactive trigger rule -->
-
 ## Rule Details
 
 This rule enforces to use interactive elements as tooltip triggers. Interactive elements can be Primer `Button`, `IconButton` and `Link` components or native elements like `button`, `a` with an `href` attribute, `select`, `textarea`, `summary` and `input` (that is not a `hidden` type).

src/configs/recommended.js

@@ -10,7 +10,8 @@ module.exports = {
   rules: {
     'primer-react/direct-slot-children': 'error',
     'primer-react/no-deprecated-colors': 'warn',
-    'primer-react/no-system-props': 'warn'
+    'primer-react/no-system-props': 'warn',
+    'primer-react/a11y-tooltip-non-interactive-trigger': 'error'
   },
   settings: {
     github: {

src/index.js

@@ -2,7 +2,8 @@ module.exports = {
   rules: {
     'direct-slot-children': require('./rules/direct-slot-children'),
     'no-deprecated-colors': require('./rules/no-deprecated-colors'),
-    'no-system-props': require('./rules/no-system-props')
+    'no-system-props': require('./rules/no-system-props'),
+    'a11y-tooltip-non-interactive-trigger': require('./rules/a11y-tooltip-non-interactive-trigger')
   },
   configs: {
     recommended: require('./configs/recommended')

src/rules/a11y-tooltip-non-interactive-trigger.js

@@ -124,11 +124,6 @@ const checkTriggerElement = jsxNode => {
 module.exports = {
   meta: {
     type: 'problem',
-    docs: {
-      url:
-        'https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/tree/HEAD/docs/rules/no-noninteractive-element-interactions.md',
-      description: 'Non-interactive elements should not be assigned mouse or keyboard event listeners.'
-    },
     schema: [
       {
         properties: {