-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement new wiz.res()
API
#41
base: main
Are you sure you want to change the base?
Conversation
sigfigs: Union[int, None] = None, | ||
decimal_places: Union[int, None] = None, | ||
) -> PrintableResult: | ||
return res(name, value, [(sys, "sys"), (stat, "stat")], unit, sigfigs, decimal_places) | ||
): |
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.
Why do you not return PrintableResult
anymore?
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 do return it in line 94. Make sure to check out the code locally to see the new contents.
It's just the GitHub diff that shows this in a weird way.
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.
@paul019 Did you find the statement?
This fixes #33 and implements the new proposal for the
wiz.res()
API.Even though it's not in the scope of this PR, we fix small errors, see 6ef4c44.
For reviewers
plum
was removed as dependency, you may want to runpipenv install
again to uninstall it on your local machine as well.