Bohdan Yefimenko comments

Results 8 comments of


                                            Bohdan Yefimenko

(fix) can go next if in center mode

merge this, plz

Rule enhancement: allow regex in forbid-component-props allowedFor / disallowedFor

@akulsr0, maybe I didn't get it, but how does `propNamePattern` solve the initially described case with `allowedFor`/`dissalowedFor`?

fix(query-core): handle errors that occur inside `setData` method

@TkDodo Also, currently, after the second fetch, the error will be `maximum call stack size exceeded.` So, probably, we need to check if the data is serializable or not, rather...

[New] `forbid-component-props`: add `allowedForPatterns` and `disallowedForPatterns` options

@ljharb what do you think?

[New] `forbid-component-props`: add `allowedForPatterns` and `disallowedForPatterns` options

Thanks a lot for the feedback, I will apply the changes.

[New] `forbid-component-props`: add `allowedForPatterns` and `disallowedForPatterns` options

Honestly, not sure what to do with the "Automatic Rebase" action, didn't find any info in [CONTRIBUTING.md](https://github.com/jsx-eslint/eslint-plugin-react/blob/master/CONTRIBUTING.md)

[New] `forbid-component-props`: add `allowedForPatterns` and `disallowedForPatterns` options

> Should there also be examples of using `propNamePatterns` with one of these new options? I've added one: https://github.com/jsx-eslint/eslint-plugin-react/pull/3805/files#diff-1cf5e390e9444274ee8064c64a72965facfa2171c5453a130f0329e57fe991adR79-R83 Also, I've added one for a more complex setup: https://github.com/jsx-eslint/eslint-plugin-react/pull/3805/files#diff-1cf5e390e9444274ee8064c64a72965facfa2171c5453a130f0329e57fe991adR106-R113 (Relates...

[New] `forbid-component-props`: add `allowedForPatterns` and `disallowedForPatterns` options

For the future, correct me if I'm wrong, but here, `rebase` is a preferable option over `merge`.