-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
--no-truncate
flag in show command
#9580
--no-truncate
flag in show command
#9580
Conversation
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 not just set width = math.inf if self.option("--no-truncate") else ...
? simplifies the code changes a lot.
Also, you might need to re-do the cli doc changes, since we reodered that doc. |
@ch1nq is this something you'd like to continue working on? |
Thanks for the review - I will apply the suggested changes |
@abn I have gone with |
36b9e22
to
ff790e6
Compare
ff790e6
to
0d4fc3a
Compare
Fixes #9597
In order to get a more consistent output from the
poetry show
command, this PR introduces a--no-truncate
flag to the command. When the flag is set the show command does not take the width of the terminal into account when deciding the level of detail to print.Pull Request Check List