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

remove special cases for setuptools and wheel #8600

Conversation

dimbleby
Copy link
Contributor

poetry install --sync will now remove those packages, if they are not required by the project being managed.

consistent with the modern world of isolated builds, #7803, etc: continuing to remove special cases for setuptools and wheel.

`poetry install --sync` will now remove those packages, if they are not
required by the project being managed.
@radoering radoering merged commit abd993c into python-poetry:master Nov 1, 2023
20 checks passed
@dimbleby dimbleby deleted the uninstall-setuptool-and-wheel-at-sync branch November 1, 2023 08:42
@johnthagen
Copy link
Contributor

johnthagen commented Nov 11, 2023

Note, this is likely an "issue" with PyCharm, but for your awareness 1.7.0 is currently pretty disruptive for PyCharm users because PyCharm assumes that setuptools will be in the environment on Python <3.12. poetry install --sync removing setuptools pops up a bunch of error messages within the IDE.

@dimbleby
Copy link
Contributor Author

you can always add setuptools as a dev dependency, which apparently for you it is

MrGreenTea pushed a commit to MrGreenTea/poetry that referenced this pull request Dec 18, 2023
Copy link

github-actions bot commented Mar 3, 2024

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 Mar 3, 2024
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.

3 participants