The issue has been solved

The `regex-shorthand` rule has a false-negative #157

sindresorhus posted onGitHub

This doesn't cause a lint error:

const foo = /^by @([a-zA-Z0-9-]+)/

Even though it could be:

const foo = /^by @([a-zA-Z\d-]+)/

I think we might need to use some more advanced detection. Maybe regexp-tree. Although, it has an optimizer API already, which I think already handles this and many other cases.

// @SamVerschueren

@issuehunt has funded $30.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

posted by IssueHuntBot about 6 years ago

This is up for grabs again. If anyone wants to tackle this, you can probably continue where https://github.com/sindresorhus/eslint-plugin-unicorn/pull/261 left off.

posted by sindresorhus over 5 years ago