feat(rules): add auto-fix support to multiple rules#30
Merged
Conversation
Add fixers to block-scope-case, catch-error-name, comment-syntax, and lines-around-comment rules: - block-scope-case: wrap case consequent in braces - catch-error-name: rename the catch parameter and all its references, bailing out when the expected name is already used - comment-syntax: fix line/block comment spacing, capitalization, list items, paragraph endings, and JSDoc formatting - lines-around-comment: apply fix for missing blank lines Add test coverage for the new fix behavior.
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds auto-fix support to the four remaining rules that previously only reported problems, so violations can now be resolved with
eslint --fix:block-scope-case: wraps case consequents in bracescatch-error-name: renames the catch parameter and all its references; bails out if the expected name is already in scope to avoid shadowingcomment-syntax: fixes spacing, capitalization, list items, paragraph endings, and JSDoc formattinglines-around-comment: inserts missing blank linesNo changes to what the rules report — only fixers were added, with tests covering the new fix output.