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

Compatibility with isort 5.x #15

Closed
wants to merge 4 commits into from
Closed

Compatibility with isort 5.x #15

wants to merge 4 commits into from

Conversation

akaihola
Copy link
Owner

@akaihola akaihola commented Jul 4, 2020

Let's use #16 instead and discard this PR. But I'll keep this open for now so we can easily see the diff in case there's something useful there.

@akaihola akaihola requested a review from Mystic-Mirage July 4, 2020 17:31
@akaihola akaihola changed the base branch from master to isort_config July 4, 2020 17:32
@akaihola
Copy link
Owner Author

akaihola commented Jul 4, 2020

I based this on top of isort_config so the diff only shows changes on top of that.

@Mystic-Mirage
Copy link
Collaborator

@akaihola I don't want to #13 to be merged before isort>5 because new isort allows writing less of code.

@akaihola akaihola closed this Jul 5, 2020
@akaihola akaihola deleted the isort5 branch July 5, 2020 10:56
@akaihola akaihola added bug Something isn't working duplicate This issue or pull request already exists labels Jul 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants