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.
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
Docs: wrong example code for "exceptions" option in id-length rule #6397
Comments
Hi @cristian-sima, thank you for the issue! |
@cristian-sima Good eyes. Welcome to ESLint! Yes, it is a copy-and-paste error.
Therefore, the option needs to be @cristian-sima Here are the Contributing Guidelines: http://eslint.org/docs/developer-guide/contributing/ |
ping @cristian-sima just checking if you would be willing to work on this one. No problem at all if you can't, just let us know :) |
I'm looking to contribute my first PR to gain some experience. From what I see this issue seems possible for me to do and tagged beginner. May I try? ( if so, can I get a little helping hand? :-) ) Thanks! |
Hi @GramParallelo, thanks for stepping up! It looks like @kaicataldo might be working on this- hopefully he can clarify whether he is or not. If you're looking for a good first pull request area, try filtering the issues to include the "beginner" label. If you can't see one, feel free to visit us in our chat room. |
I'm not working on this - just labeled it as I thought it would be a great issue for a newcomer to the project to start with :) Thanks for contributing, @GramParallelo! |
@GramParallelo this comment should help you get started, let us know if you need any help! |
Changed 2 typos: exceptions accepts array, was max property instead of exception property.
Changed 2 typos: exceptions accepts array, was max property instead of exception property.
Changed 2 typos: exceptions accepts array, was max property instead of exception property.
Changed 2 typos: exceptions accepts array, was max property instead of exception property.
http://eslint.org/docs/rules/id-length#exceptions
It is taking about exceptions, but there is a max property
The text was updated successfully, but these errors were encountered: