Skip to content

fix: add cache path in the error#5091

Merged
baszalmstra merged 4 commits intoprefix-dev:mainfrom
nichmor:fix/unclear-pypi-error-message
Dec 9, 2025
Merged

fix: add cache path in the error#5091
baszalmstra merged 4 commits intoprefix-dev:mainfrom
nichmor:fix/unclear-pypi-error-message

Conversation

@nichmor
Copy link
Contributor

@nichmor nichmor commented Dec 9, 2025

Description

Fixes: #4941

In this PR we are adding the location from where we tried to fetch the cache file in the error.

How Has This Been Tested?

Using an integration test.

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.

Tools: Claude

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added sufficient tests to cover my changes.
  • I have verified that changes that would impact the JSON schema have been made in schema/model.py.

@nichmor nichmor marked this pull request as ready for review December 9, 2025 10:00
Copy link
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

Great improvement

@baszalmstra baszalmstra merged commit dcb8d02 into prefix-dev:main Dec 9, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unclear error message when dealing with pypi dependencies

3 participants