search

jsx-eslint / eslint-plugin-react

React-specific linting rules for ESLint
MIT License
8.86k stars 2.76k forks source link

[Enhance] forbid-component-props accepts allowedForRegex and disallowedForRegex #3687

Open bschlenk opened 4 months ago

bschlenk commented 4 months ago

Fixes https://github.com/jsx-eslint/eslint-plugin-react/issues/3686

codecov[bot] commented 4 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (9f4b2b9) 97.74% compared to head (7723121) 97.73%.

Files Patch % Lines
lib/rules/forbid-component-props.js 93.33% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3687 +/- ## ========================================== - Coverage 97.74% 97.73% -0.01% ========================================== Files 132 132 Lines 9409 9420 +11 Branches 3449 3454 +5 ========================================== + Hits 9197 9207 +10 - Misses 212 213 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ljharb commented 4 months ago

PR was a bit premature; regexes in eslint configs are hugely insecure and a CVE farm, and should never exist.