-
Notifications
You must be signed in to change notification settings - Fork 0
feat: ✨ re-export print() from rich, use it explain() docs
#265
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
Conversation
…package into feat/reexport-rich-print-for-explain
joelostblom
left a comment
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.
Great idea! A couple of suggestions but none required if you disagree.
signekb
left a comment
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.
Nice! Some comments and suggestions:
Co-authored-by: Signe Kirk Brødbæk <[email protected]>
…package into feat/reexport-rich-print-for-explain
signekb
left a comment
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.
Just one thing, then I'll approve
Co-authored-by: Signe Kirk Brødbæk <[email protected]>
Description
Closes #263
Needs a quick review.
Checklist
just run-all