Skip to content

Config fix for eslint react/jsx-no-duplicate-props #7054

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
rommguy opened this issue May 15, 2019 · 2 comments
Closed

Config fix for eslint react/jsx-no-duplicate-props #7054

rommguy opened this issue May 15, 2019 · 2 comments

Comments

@rommguy
Copy link
Contributor

rommguy commented May 15, 2019 •
edited
Loading

First - thanks for this amazing tool, it's one of the best tools I've used

Bug report -
The eslint rules here have a config problem with react/jsx-no-duplicate-props - ignoreCase param should be false.
Some APIs like Material UI TextField allow different props with case differences only. In the Material UI example a TextField can get inputProps for the input dom node, and InputProps for the Input component.
So the config for this rule should have ignoreCase: false

I'll be happy to create a PR for this
Thanks
@nmain
Copy link

nmain commented May 15, 2019

See #6956

@rommguy rommguy closed this as completed May 15, 2019
@rommguy
Copy link
Contributor Author

rommguy commented May 15, 2019

Closed - duplicate

@lock lock bot locked and limited conversation to collaborators May 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rommguy @nmain