Repository: lo1tuma/eslint-plugin-mocha
Pull request #229 (2020-02-17 15:30:24)

Indicate whether rule is recommended
Builds on #228

- Indicate whether rule is recommended (also put into table along with info on whether "fixable")


lo1tuma: #228 has been merged, can you rebase and fix the failing test?

Patch:
diff --git a/docs/rules/README.md b/docs/rules/README.md
index 7b796e3..bb2ac38 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -1,23 +1,25 @@
 # Rules
 
-* [handle-done-callback](handle-done-callback.md) - enforces handling of callbacks for async tests
-* [max-top-level-suites](max-top-level-suites.md) - limit the number of top-level suites in a single file
-* [no-async-describe](no-async-describe.md) - disallow async functions passed to describe (fixable)
-* [no-exclusive-tests](no-exclusive-tests.md) - disallow exclusive mocha tests
-* [no-global-tests](no-global-tests.md) - disallow global tests
-* [no-hooks](no-hooks.md) - disallow hooks
-* [no-hooks-for-single-case](no-hooks-for-single-case.md) - disallow hooks for a single test or test suite
-* [no-identical-title](no-identical-title.md) - disallow identical titles
-* [no-mocha-arrows](no-mocha-arrows.md) - disallow arrow functions as arguments to mocha globals (fixable)
-* [no-nested-tests](no-nested-tests.md) - disallow tests to be nested within other tests
-* [no-pending-tests](no-pending-tests.md) - disallow pending/unimplemented mocha tests
-* [no-return-and-callback](no-return-and-callback.md) - disallow returning in a test or hook function that uses a callback
-* [no-return-from-async](no-return-from-async.md) - disallow returning from an async test or hook
-* [no-setup-in-describe](no-setup-in-describe.md) - disallow calling functions and dot operators directly in describe blocks
-* [no-sibling-hooks](no-sibling-hooks.md) - disallow duplicate uses of a hook at the same level inside a describe
-* [no-skipped-tests](no-skipped-tests.md) - disallow skipped mocha tests (fixable)
-* [no-synchronous-tests](no-synchronous-tests.md) - disallow synchronous tests
-* [no-top-level-hooks](no-top-level-hooks.md) - disallow top-level hooks
-* [prefer-arrow-callback](prefer-arrow-callback.md) - prefer arrow function callbacks (mocha-aware) (fixable)
-* [valid-suite-description](valid-suite-description.md) - match suite descriptions against a pre-configured regular expression
-* [valid-test-description](valid-test-description.md) - match test descriptions against a pre-configured regular expression
+|recommended|fixable|rule|description|
+|-|-|-|-|
+|:heavy_check_mark:|| [handle-done-callback](handle-done-callback.md) | enforces handling of callbacks for async tests
+|:heavy_check_mark:|| [max-top-level-suites](max-top-level-suites.md) | limit the number of top-level suites in a single file
+|:heavy_check_mark:|:wrench:| [no-async-describe](no-async-describe.md) | disallow async functions passed to describe
+|:heavy_check_mark:|| [no-exclusive-tests](no-exclusive-tests.md) | disallow exclusive mocha tests
+|:heavy_check_mark:|| [no-global-tests](no-global-tests.md) | disallow global tests
+||| [no-hooks](no-hooks.md) | disallow hooks
+|:heavy_check_mark:|| [no-hooks-for-single-case](no-hooks-for-single-case.md) | disallow hooks for a single test or test suite
+|:heavy_check_mark:|| [no-identical-title](no-identical-title.md) | disallow identical titles
+|:heavy_check_mark:|:wrench:| [no-mocha-arrows](no-mocha-arrows.md) | disallow arrow functions as arguments to mocha globals
+|:heavy_check_mark:|| [no-nested-tests](no-nested-tests.md) | disallow tests to be nested within other tests
+|:heavy_check_mark:|| [no-pending-tests](no-pending-tests.md) | disallow pending/unimplemented mocha tests
+|:heavy_check_mark:|| [no-return-and-callback](no-return-and-callback.md) | disallow returning in a test or hook function that uses a callback
+||| [no-return-from-async](no-return-from-async.md) | disallow returning from an async test or hook
+|:heavy_check_mark:|| [no-setup-in-describe](no-setup-in-describe.md) | disallow calling functions and dot operators directly in describe blocks
+|:heavy_check_mark:|| [no-sibling-hooks](no-sibling-hooks.md) | disallow duplicate uses of a hook at the same level inside a describe
+|:heavy_check_mark:|:wrench:| [no-skipped-tests](no-skipped-tests.md) | disallow skipped mocha tests
+||| [no-synchronous-tests](no-synchronous-tests.md) | disallow synchronous tests
+|:heavy_check_mark:|| [no-top-level-hooks](no-top-level-hooks.md) | disallow top-level hooks
+||:wrench:| [prefer-arrow-callback](prefer-arrow-callback.md) | prefer arrow function callbacks (mocha-aware)
+||| [valid-suite-description](valid-suite-description.md) | match suite descriptions against a pre-configured regular expression
+||| [valid-test-description](valid-test-description.md) | match test descriptions against a pre-configured regular expression


Test patch:
diff --git a/test/index.js b/test/index.js
index fc02466..103486d 100644
--- a/test/index.js
+++ b/test/index.js
@@ -60,7 +60,7 @@ describe('eslint-plugin-mocha', function () {
         it('should be linked in the documentation index', function () {
             documentationFiles.forEach(function (file) {
                 const ruleName = path.basename(file, '.md');
-                const expectedLink = `* [${ ruleName }](${ file })`;
+                const expectedLink = `| [${ ruleName }](${ file }) |`;
 
                 expect(documentationIndex).to.contain(expectedLink);
             });