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

Disallow unmatched opening square bracket in regular expressions #176

Conversation

baryla
Copy link

@baryla baryla commented Mar 15, 2020

The rule does pick out the opening square bracket but for some reason, the error underline is not right?

Screenshot 2020-03-15 at 15 46 30

Btw. sorry about the long lint name 😂I'm open to better suggestions 😂

@facebook-github-bot
Copy link

Hi @baryla!

Thank you for your pull request and welcome to our community.We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.

In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

@baryla baryla changed the title Lint/no unmatched opening square bracket in regular expressions Disallow unmatched opening square bracket in regular expressions Mar 15, 2020
@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@baryla
Copy link
Author

baryla commented Mar 15, 2020

As pointed out by @KevinKelbie in the discord channel, there's some more work that needs to be done.

@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@Kelbie Kelbie mentioned this pull request Mar 19, 2020
23 tasks
@sebmck
Copy link
Contributor

sebmck commented Apr 24, 2020

@baryla

As pointed out by @KevinKelbie in the discord channel, there's some more work that needs to be done.

I think I missed that conversation. What's remaining?

@Kelbie
Copy link
Contributor

Kelbie commented May 13, 2020

@sebmck sebmck changed the base branch from master to main June 12, 2020 03:57
@sebmck
Copy link
Contributor

sebmck commented Jun 28, 2020

Closing this out due to inactivity. Feel free to open a new PR if you'd like to pick this up again! Thank you!

@sebmck sebmck closed this Jun 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants