Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

noControlCharactersInRegex, no-control-regex #3962

Closed
Tracked by #3953
ematipico opened this issue Dec 5, 2022 · 6 comments · Fixed by #4656
Closed
Tracked by #3953

noControlCharactersInRegex, no-control-regex #3962

ematipico opened this issue Dec 5, 2022 · 6 comments · Fixed by #4656
Assignees
@unvalley
Labels
A-Linter Area: linter I-Normal Implementation: normal understanding of the tool and awareness

Comments

@ematipico
Copy link
Contributor

ematipico commented Dec 5, 2022 •
edited
Loading

https://eslint.org/docs/latest/rules/no-control-regex
@ematipico ematipico mentioned this issue Dec 5, 2022
Open
30 tasks
@ematipico ematipico added the I-Normal Implementation: normal understanding of the tool and awareness label Dec 5, 2022
@ematipico ematipico changed the title https://eslint.org/docs/latest/rules/no-control-regex noControlCharactersInRegex Dec 5, 2022
@ematipico
Copy link
Contributor Author

I propose the name noControlCharactersInRegex, the name of the ESLint and its description were a bit misleading.

@ematipico ematipico changed the title noControlCharactersInRegex noControlCharactersInRegex, no-control-regex Dec 5, 2022
@mzbac
Copy link
Contributor

mzbac commented Dec 12, 2022

I would like to pick up this. Let me know if you have any concerns.

@mzbac mzbac mentioned this issue Dec 16, 2022
Closed
2 tasks
@github-actions
Copy link

👋 @rome/staff please triage this issue by adding one of the following labels: S-Bug: confirmed, S-Planned , S-Wishlist or umbrella

@ematipico ematipico removed the S-Stale label Feb 14, 2023
@Conaclos Conaclos added the A-Linter Area: linter label Feb 22, 2023
@ematipico
Copy link
Contributor Author

@mzbac are you still interested at contributing to this rule?

@mzbac
Copy link
Contributor

mzbac commented Apr 5, 2023

@ematipico Sorry, I thought this was already resolved. If no one would like to pick up this task, I don't mind working on it.

@unvalley unvalley assigned unvalley and unassigned mzbac Jun 30, 2023
@unvalley
Copy link
Contributor

@mzbac Thanks for making the PR before. I will make a PR as a follow up to that.

@unvalley unvalley mentioned this issue Jul 4, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@unvalley unvalley

Labels
A-Linter Area: linter I-Normal Implementation: normal understanding of the tool and awareness
Projects
ESlint recommended rules
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(rome_js_analyze): noControlCharactersInRegex
4 participants
@ematipico @Conaclos @mzbac @unvalley