-
Notifications
You must be signed in to change notification settings - Fork 65
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
flatMap not detected as part of a lodash chain #49
Comments
The lodash docs are a bit fuzzy on the matter (there's separate docs for |
Even things the docs say arent possible e.g. join return a wrapper. We will see what jdalton says in a bit anyway.. |
I answered you by myself in the issue. The docs are unclear about |
So now I understand - it looks to me like there is another bug - the lodash utils exits early if the method is not chainable, without checking to see if its implicit or explicit... |
Right again! I'll open a separate issue. |
Sync chainable in v4 with lodash master. Fixes #49
this
is hitting prefer-lodash-method because
flatMap
isn't marked chainable.But am also confused by the lodash documentation - I raised this..
lodash/lodash#2166
because it looks like everything is chainable.. so am not sure whether everything should be chainable or just to add
flatMap
to the chainable list..The text was updated successfully, but these errors were encountered: