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

Depend on setuptools #1875

Closed
wants to merge 1 commit into from
Closed

Depend on setuptools #1875

wants to merge 1 commit into from

Conversation

m-kuhn
Copy link

@m-kuhn m-kuhn commented Dec 28, 2021

Fixes #1874

jsf9k added a commit to cisagov/log4j-md-yml that referenced this pull request Dec 28, 2021
This PR resolves PyCQA/isort#1874, which is currently biting us when
we attempt to install the isort pre-commit hook.
@timothycrosley
Copy link
Member

This is not an isort specific issue. isort hasn't made any releases that change this behavior, and in fact hasn't made any release in some months. It also has no external dependencies. isort uses poetry intentionally and not setuptools, this change would misrepresent how isort should be installed.This breakage is occurring upstream and I encourage you to reach out to the projects responsible, and in the meantime utilize SETUPTOOLS_USE_DISTUTILS=stdlib or one of the other workarounds @asottile has helpfully cataloged here: pre-commit/pre-commit#2178 (comment)

mcdonnnj added a commit to cisagov/log4j-md-yml that referenced this pull request Dec 29, 2021
jsf9k pushed a commit to cisagov/log4j-md-yml that referenced this pull request Dec 29, 2021
jsf9k pushed a commit to cisagov/log4j-md-yml that referenced this pull request Dec 29, 2021
This reverts commit 51752ee.

The workaround added in that commit is no longer required, since the
issue was fixed in setuptools 60.1.1.
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.

No module named 'setuptools' when installing through pre-commit
3 participants