Skip to content

Conversation

@mshafer-NI
Copy link
Collaborator

I've learned that isort now has a black profile allowing it to play nicely with black.

To start investigating bringing flake8-isort in, I opted to first test what configurations we would need to match our current style.

Changing to configuring the call to isort within _sort_imports removes the need for the custom logic around splitting multi-import lines, so choosing to contribute this simplification at this point.

@mshafer-NI mshafer-NI merged commit 478eebd into ni:main Oct 6, 2022
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.

2 participants