Skip to content

Conversation

@elliot-100
Copy link
Contributor

@elliot-100 elliot-100 commented Oct 10, 2024

I feel that from a user's POV, poetry show has two fairly separate modes, 'list packages' and 'single package info'.

  • This PR separates them in documentation, which I feel is clearer.

    For example it shows that 'single package info' only has one useful option, --tree --why.

    (I think the code should probably be refactored to better separate the cases and drop a few interactions between but that's one for another day.)

  • It adds missing example for the `list packages' mode.

  • It explains what 'poetry show' actually tells you, resolving my very old discussion question What does `poetry show` actually tell me? #7311.

  • It also improves ordering of options and wording.

  • It does not attempt to cover all the colour coding that may appear in show output. To be honest it's inacesssible design to use colour alone to convey information.

[TODO: check CLI --help text and error messages]

Pull Request Check List

Resolves: #7311

  • Added tests for changed code -- not relevant for docs-only change
  • Updated documentation for changed code -- not relevant for docs-only change

If the terminal supports coloured text, packages that aren't installed are shown with their names in red.

If the terminal doesn't support coloured text, packages that aren't installed are shown with their version prefixed
with `(!)`.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
with `(!)`.
with `(!)`.

@elliot-100 elliot-100 closed this Mar 15, 2025
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants