Skip to content

Commit ff9fb70

Browse files
ZamiellZamiell
committed
refactor(plugin)!: clean up complete rules
1 parent c141121 commit ff9fb70

File tree

126 files changed

+7
-13765
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

126 files changed

+7
-13765
lines changed

‎packages/eslint-plugin-isaacscript/README.md

Lines changed: 6 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -35,52 +35,12 @@ Each rule has emojis denoting:
3535
<!-- Do not manually modify the RULES_TABLE section. Instead, run: npm run generate -->
3636
<!-- RULES_TABLE -->
3737

38-
| Name | Description | :white_check_mark: | :wrench: | :thought_balloon: |
39-
| -------------------------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------- | ------------------ | -------- | ----------------- |
40-
| [`isaacscript/complete-sentences-jsdoc`](docs/rules/complete-sentences-jsdoc.md) | Requires complete sentences for JSDoc comments | :white_check_mark: | | |
41-
| [`isaacscript/complete-sentences-line-comments`](docs/rules/complete-sentences-line-comments.md) | Requires complete sentences for multi-line leading line comments | :white_check_mark: | | |
42-
| [`isaacscript/consistent-enum-values`](docs/rules/consistent-enum-values.md) | Requires consistent enum values | :white_check_mark: | | |
43-
| [`isaacscript/consistent-named-tuples`](docs/rules/consistent-named-tuples.md) | Requires that if one or more tuple elements are named, all of them are named | :white_check_mark: | | |
44-
| [`isaacscript/enum-member-number-separation`](docs/rules/enum-member-number-separation.md) | Disallows numbers next to letters in enum members | | | |
45-
| [`isaacscript/eqeqeq-fix`](docs/rules/eqeqeq-fix.md) | Requires the use of `===` and `!==` (and automatically fixes) | :white_check_mark: | :wrench: | |
46-
| [`isaacscript/format-jsdoc-comments`](docs/rules/format-jsdoc-comments.md) | Disallows `/**` comments longer than N characters and multi-line comments that can be merged together | :white_check_mark: | :wrench: | |
47-
| [`isaacscript/format-line-comments`](docs/rules/format-line-comments.md) | Disallows `//` comments longer than N characters and multi-line comments that can be merged together | :white_check_mark: | :wrench: | |
48-
| [`isaacscript/jsdoc-code-block-language`](docs/rules/jsdoc-code-block-language.md) | Requires a language specification for every JSDoc code block | :white_check_mark: | | |
49-
| [`isaacscript/newline-between-switch-case`](docs/rules/newline-between-switch-case.md) | Requires newlines between switch cases | :white_check_mark: | :wrench: | |
50-
| [`isaacscript/no-confusing-set-methods`](docs/rules/no-confusing-set-methods.md) | Disallows confusing methods for sets | :white_check_mark: | | :thought_balloon: |
51-
| [`isaacscript/no-empty-jsdoc`](docs/rules/no-empty-jsdoc.md) | Disallows empty JSDoc comments | :white_check_mark: | :wrench: | |
52-
| [`isaacscript/no-empty-line-comments`](docs/rules/no-empty-line-comments.md) | Disallows empty line comments | :white_check_mark: | :wrench: | |
53-
| [`isaacscript/no-explicit-array-loops`](docs/rules/no-explicit-array-loops.md) | Disallows explicit iteration for arrays | :white_check_mark: | :wrench: | :thought_balloon: |
54-
| [`isaacscript/no-explicit-map-set-loops`](docs/rules/no-explicit-map-set-loops.md) | Disallows explicit iteration for maps and sets | :white_check_mark: | :wrench: | :thought_balloon: |
55-
| [`isaacscript/no-for-in`](docs/rules/no-for-in.md) | Disallows "for x in y" statements | :white_check_mark: | | |
56-
| [`isaacscript/no-invalid-default-map`](docs/rules/no-invalid-default-map.md) | Disallows invalid constructors for the DefaultMap class | | | :thought_balloon: |
57-
| [`isaacscript/no-let-any`](docs/rules/no-let-any.md) | Disallows declaring variables with let that do not have a type | :white_check_mark: | | :thought_balloon: |
58-
| [`isaacscript/no-mutable-return`](docs/rules/no-mutable-return.md) | Disallows returning mutable arrays, maps, and sets from functions | :white_check_mark: | | :thought_balloon: |
59-
| [`isaacscript/no-number-enums`](docs/rules/no-number-enums.md) | Disallows number enums | :white_check_mark: | | |
60-
| [`isaacscript/no-object-any`](docs/rules/no-object-any.md) | Disallows declaring objects and arrays that do not have a type | :white_check_mark: | | :thought_balloon: |
61-
| [`isaacscript/no-object-methods-with-map-set`](docs/rules/no-object-methods-with-map-set.md) | Disallows using object methods with maps and sets | :white_check_mark: | | :thought_balloon: |
62-
| [`isaacscript/no-string-length-0`](docs/rules/no-string-length-0.md) | Disallows checking for empty strings via the length method in favor of direct comparison to an empty string | :white_check_mark: | | :thought_balloon: |
63-
| [`isaacscript/no-template-curly-in-string-fix`](docs/rules/no-template-curly-in-string-fix.md) | Disallows template literal placeholder syntax in regular strings (and automatically fixes) | :white_check_mark: | :wrench: | |
64-
| [`isaacscript/no-throw`](docs/rules/no-throw.md) | Disallows the usage of "throw" | | | |
65-
| [`isaacscript/no-undefined-return-type`](docs/rules/no-undefined-return-type.md) | Disallows `undefined` return types on functions | :white_check_mark: | | :thought_balloon: |
66-
| [`isaacscript/no-unnecessary-assignment`](docs/rules/no-unnecessary-assignment.md) | Disallows useless assignments | :white_check_mark: | | :thought_balloon: |
67-
| [`isaacscript/no-unsafe-plusplus`](docs/rules/no-unsafe-plusplus.md) | Disallow unsafe and confusing uses of the "++" and "--" operators | :white_check_mark: | | :thought_balloon: |
68-
| [`isaacscript/no-useless-return`](docs/rules/no-useless-return.md) | Disallow redundant return statements (with no autofixer) | :white_check_mark: | | |
69-
| [`isaacscript/no-void-return-type`](docs/rules/no-void-return-type.md) | Disallows `void` return types on non-exported functions | :white_check_mark: | :wrench: | |
70-
| [`isaacscript/prefer-const`](docs/rules/prefer-const.md) | Require `const` declarations for variables that are never reassigned after declared (with no autofixer) | :white_check_mark: | | |
71-
| [`isaacscript/prefer-plusplus`](docs/rules/prefer-plusplus.md) | Require "++" or "--" operators instead of assignment operators where applicable | :white_check_mark: | :wrench: | |
72-
| [`isaacscript/prefer-postfix-plusplus`](docs/rules/prefer-postfix-plusplus.md) | Require "i++" instead of "++i" | :white_check_mark: | | :thought_balloon: |
73-
| [`isaacscript/prefer-readonly-parameter-types`](docs/rules/prefer-readonly-parameter-types.md) | Require function parameters to be typed as `readonly` to prevent accidental mutation of inputs | :white_check_mark: | | :thought_balloon: |
74-
| [`isaacscript/require-break`](docs/rules/require-break.md) | Requires that each case of a switch statement has a `break` statement | :white_check_mark: | | |
75-
| [`isaacscript/require-capital-const-assertions`](docs/rules/require-capital-const-assertions.md) | Requires a capital letter for named objects and arrays that have a const assertion | :white_check_mark: | :wrench: | |
76-
| [`isaacscript/require-capital-read-only`](docs/rules/require-capital-read-only.md) | Requires maps/sets/arrays with a capital letter to be read-only | :white_check_mark: | | :thought_balloon: |
77-
| [`isaacscript/require-unannotated-const-assertions`](docs/rules/require-unannotated-const-assertions.md) | Disallows explicit type annotations for variables that have a const assertion | :white_check_mark: | | |
78-
| [`isaacscript/require-v-registration`](docs/rules/require-v-registration.md) | Require variables named "v" to be registered with the save data manager | | | |
79-
| [`isaacscript/require-variadic-function-argument`](docs/rules/require-variadic-function-argument.md) | Requires that variadic functions must be supplied with at least one argument | :white_check_mark: | | :thought_balloon: |
80-
| [`isaacscript/strict-array-methods`](docs/rules/strict-array-methods.md) | Requires boolean return types on array method functions | :white_check_mark: | | :thought_balloon: |
81-
| [`isaacscript/strict-enums`](docs/rules/strict-enums.md) | Disallows the usage of unsafe enum patterns | :white_check_mark: | | :thought_balloon: |
82-
| [`isaacscript/strict-undefined-functions`](docs/rules/strict-undefined-functions.md) | Disallows empty return statements in functions annotated as returning undefined | :white_check_mark: | | :thought_balloon: |
83-
| [`isaacscript/strict-void-functions`](docs/rules/strict-void-functions.md) | Disallows non-empty return statements in functions annotated as returning void | :white_check_mark: | | |
38+
| Name | Description | :white_check_mark: | :wrench: | :thought_balloon: |
39+
| ------------------------------------------------------------------------------------------ | ----------------------------------------------------------------------- | ------------------ | -------- | ----------------- |
40+
| [`isaacscript/enum-member-number-separation`](docs/rules/enum-member-number-separation.md) | Disallows numbers next to letters in enum members | | | |
41+
| [`isaacscript/no-invalid-default-map`](docs/rules/no-invalid-default-map.md) | Disallows invalid constructors for the DefaultMap class | | | :thought_balloon: |
42+
| [`isaacscript/no-throw`](docs/rules/no-throw.md) | Disallows the usage of "throw" | | | |
43+
| [`isaacscript/require-v-registration`](docs/rules/require-v-registration.md) | Require variables named "v" to be registered with the save data manager | | | |
8444

8545
<!-- /RULES_TABLE -->
8646

‎packages/eslint-plugin-isaacscript/docs/rules/complete-sentences-jsdoc.md

Lines changed: 0 additions & 88 deletions
This file was deleted.

‎packages/eslint-plugin-isaacscript/docs/rules/complete-sentences-line-comments.md

Lines changed: 0 additions & 29 deletions
This file was deleted.

‎packages/eslint-plugin-isaacscript/docs/rules/consistent-enum-values.md

Lines changed: 0 additions & 65 deletions
This file was deleted.

‎packages/eslint-plugin-isaacscript/docs/rules/consistent-named-tuples.md

Lines changed: 0 additions & 31 deletions
This file was deleted.

0 commit comments

Comments
 (0)