-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Meaningful Errors On Invalid Input? #667
Comments
(I'm mostly just helping out with docs, but to clarify:) Would you expect it to still throw an error in the same place but with a more helpful message? |
Right. Would expect it to throw an error, just something more meaningful. Something like On Fri, Nov 21, 2014 at 5:42 PM, Beau Gunderson notifications@github.com
|
Another option would be to treat a |
👍 I'm more of a fan of being graceful about it to be honest! On Sat, Nov 22, 2014 at 6:27 PM, Alexander Early notifications@github.com
|
@jamescarr Just to be clear, are you saying you prefer the lodash approach of treating |
Still crashing, fixed adding this in line 120: |
Ran across this issue today with some user code on our site:
Is it possible to do some kind of assertion or something at the beginning of the method to ensure that invalid data is not passed in?
The text was updated successfully, but these errors were encountered: