-
Notifications
You must be signed in to change notification settings - Fork 4
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
Try to replace .expect
s with safe internal APIs
#31
Comments
Could you please provide further information and an example within the code so that we do not forget the track of this in the future? |
When searching for
I think that the only good use of
|
There are several places in the library with
.expect
s that could be eliminated. However, this is low on our list of priorities and I am just filing this issue to remind me in the future.The text was updated successfully, but these errors were encountered: