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

PR: Update installing developer repos to accommodate dist-info #22709

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

mrclary
Copy link
Contributor

@mrclary mrclary commented Oct 22, 2024

When Spyder is installed in editable mode, .dist-info is created instead of .egg-link.
This breaks the test for whether Spyder is installed in editable mode. The test is updated to accommodate .dist-info by checking for editable in the direct_url.json file in .dist-info.

@mrclary mrclary self-assigned this Oct 22, 2024
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks @mrclary for fixing this! It was a really annoying problem and the only workaround I found for it was removing pyproject.toml and then restoring it.

@ccordoba12 ccordoba12 added this to the v6.0.2 milestone Oct 24, 2024
@ccordoba12 ccordoba12 merged commit 3fcdd8a into spyder-ide:master Oct 24, 2024
17 checks passed
@ccordoba12
Copy link
Member

@meeseeksdev please backport to 6.x

meeseeksmachine pushed a commit to meeseeksmachine/spyder that referenced this pull request Oct 24, 2024
ccordoba12 pushed a commit that referenced this pull request Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants