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

Increase required isort version to 4.3.5. #76

Merged
merged 1 commit into from Jan 21, 2020

Conversation

pedrofreire
Copy link
Contributor

Following #71, pull #75 added the pyproject extra to the isort dependency. However, having 'isort[pyproject] >= 4.3.0' as a requirement may make installation fail, because isort versions 4.3.0 to 4.3.4 do not have the pyproject extra, as it was added in version 4.3.5. Increasing the minimum version to 4.3.5 should fix this.


Example of error when attempting to install flake8-isort:

pkg_resources.UnknownExtra: isort 4.3.4 has no such extra feature 'pyproject'

Having 'isort[pyproject] >= 4.3.0' as a requirement may make installation
fail, because isort versions 4.3.0 to 4.3.4 do not have the pyproject
extra, as it was added in version 4.3.5. Increasing the minimum version
to 4.3.5 should fix this.

Example of error when attempting to install flake8-isort:

pkg_resources.UnknownExtra: isort 4.3.4 has no such extra feature 'pyproject'
@gforcada
Copy link
Owner

@pedrofreire thanks for the fix!

@gforcada gforcada merged commit 8af7b8f into gforcada:master Jan 21, 2020
bnavigator added a commit to bnavigator/flake8-isort that referenced this pull request Jul 13, 2020
Isort > 5 does not have the pyproject extra either.
Just require toml directly for the tests
bnavigator added a commit to bnavigator/flake8-isort that referenced this pull request Jul 13, 2020
Isort > 5 does not have the pyproject extra either.
Just require toml directly for the tests
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.

None yet

2 participants