-
Notifications
You must be signed in to change notification settings - Fork 120
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
Re-export errors from ops and unexport ops module #282
Conversation
This also moves errors that are specific to add to a separate file. Otherwise rustdoc would show them to be included for the features: "(add or not) and add", which is obviously equivalent to add but reads super weird.
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.
@JelteF I've moved add
module back to ops
as ops::add
one. This seems more logical to me since add
uses types from ops
, and in std
they're all united under ops
module too.
Please, take a look and merge it, if it's OK for you.
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.
@JelteF done.
Resolves #274
Synopsis
Re-export error types from the ops module from the root of the crate. And
unexport the ops module itself.
This also moves errors that are specific to add to a separate file.
Otherwise rustdoc would show them to be included for the features:
"(add or not) and add", which is obviously equivalent to add but reads
super weird.
Checklist
Documentation is updated (if required)Tests are added/updated (if required)CHANGELOG entry is added (if required)