We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I found two panic cases due to calling unwrap on None.
unwrap
let datetime = Local::now(); datetime.sub(Months::new(312345123)); // first panic datetime.sub(Days::new(312345123)); // second panic
It might be better to have panic msgs (e.g., with expect) and mention them in the documentation.
expect
The text was updated successfully, but these errors were encountered:
Add
Sub
Thank you for the report. Changed to expect and added documentation in #1316.
Sorry, something went wrong.
No branches or pull requests
I found two panic cases due to calling
unwrap
on None.It might be better to have panic msgs (e.g., with
expect
) and mention them in the documentation.The text was updated successfully, but these errors were encountered: