docs(linter): Add configuration docs to no-useless-escape lint rule.#14791
docs(linter): Add configuration docs to no-useless-escape lint rule.#14791camc314 merged 4 commits intooxc-project:mainfrom
Conversation
Also add missing text for "Why is this bad?" section.
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive documentation to the no-useless-escape lint rule, including missing content for the "Why is this bad?" section and a new "Options" section with details about the allowRegexCharacters configuration option.
- Added explanatory text for why unnecessary escapes are problematic
- Added complete Options documentation with configuration details and JSON example
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Connor Shea <connor.james.shea@gmail.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
CodSpeed Performance ReportMerging #14791 will not alter performanceComparing Summary
Footnotes
|
Also add missing text for "Why is this bad?" section. I've updated this to use the autogen docs rather than an explicit Options section, since that seems to be the preferred implementation path for this kind of addition.
Part of #14743.
Results in: