-
-
Notifications
You must be signed in to change notification settings - Fork 782
✨ Add option to set the width of the Rich exception output #528
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
base: master
Are you sure you want to change the base?
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
78eb706
to
2587d6d
Compare
Hi @tiangolo , |
I definitely would like for this to be merged in! @tiangolo |
Yes, feature looks really good. @tiangolo Could you please, review? |
adding new attribute in DeveloperExceptionConfig to allow setting width for rick traceback.
add `pretty_exceptions_width` in typer class to be used by `DeveloperConfigException` to allow setting width of traceback boxes.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
📝 Docs preview for commit d651993 at: https://ad526c53.typertiangolo.pages.dev Modified Pages |
Is the intention of this PR to handle situations when it's impossible to determine the width of the terminal screen? Currently you can specify the width of the terminal screen using Lines 74 to 75 in 0bc7cce
See in the details
I think we don't need to change anything. Probably just document this ENV variable |
add parameter as suggested feature in issue #523