Skip to content
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

Suppress unearth's collector warning regarding failed links when working with multiple indexes #3226

Open
o-moe opened this issue Oct 22, 2024 · 2 comments
Labels
⭐ enhancement Improvements for existing features

Comments

@o-moe
Copy link
Contributor

o-moe commented Oct 22, 2024

Is your feature/enhancement proposal related to a problem? Please describe.

Not a problem but low user experience imho:

When additional (private) indexes are added (default pypi is not overwritten), then commands like pdm outdated or pdm list --resolve (basically every command that iterates the configured indexes and uses unearth's collector) is showing these kind of warnings for the index that does not host the package:

Failed to collect links from http://[private index url]/simple/[package name]/: Client Error(404): Not Found

Depending on the count of default or dev dependencies, those warning are flooding the terminal before the actual command output is provided.

The warning seems to originate from here.

Describe the solution you'd like

Since it is expected that some/all packages are not available on all given indexes, these warnings should be suppressed and maybe only shown if verbosity of the command is increased.

@o-moe o-moe added the ⭐ enhancement Improvements for existing features label Oct 22, 2024
@sanmai-NL
Copy link
Contributor

Duplicate of #2442

@o-moe
Copy link
Contributor Author

o-moe commented Dec 8, 2024

@sanmai-NL I am not sure if my enhancement request is acually a duplicate of your issue - at least how I understand your request: in my context, it is expected that unearth is iterating thru the given indexes to see which provides the packages to resolve. So that's not an issue. What is unexpected from a user experience perspective are the warnings when an index does not provide the package. The warnings may lead you into thinking that something may be wrong and are clustering the terminal output prior to the actual output (like with pdm outdated)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐ enhancement Improvements for existing features
Projects
None yet
Development

No branches or pull requests

2 participants