-
Notifications
You must be signed in to change notification settings - Fork 316
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
remove remaining fatalError
s
#529
Conversation
whooa, a linter! cool! I'll update |
c2e95f1
to
7802cfe
Compare
@@ -5,10 +5,14 @@ | |||
|
|||
import Foundation | |||
|
|||
enum DateExtensionsError: Error { | |||
case notValidDateFromComponents |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I think invalidDateComponents
could be a better name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good call
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry I didn't catch this before, I just realized it might be useful to capture the date components that trigger this and spit those out to aid folks in debugging. Thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
love it, adding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done! lmk what you think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it. 🎈🐐
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Date components question (could go either way) but overall I'm very excited about removing opportunities for the SDK to crash
…nvalidDateComponents error
Addresses #394
Replaces the last few remaining
fatalError
s with better error handling.