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

isort ordering issue on Travis CI #185

Closed
cas-- opened this issue Sep 26, 2014 · 3 comments
Closed

isort ordering issue on Travis CI #185

cas-- opened this issue Sep 26, 2014 · 3 comments

Comments

@cas--
Copy link

cas-- commented Sep 26, 2014

We have encountered a very strange issue with isort where by it randomly decides to switch these two imports around but only when run in a virtualenv on Travis, we cannot replicate on local machines, it passes cleanly each and every time.

This is the diff from the Travis run, with an extract below: https://travis-ci.org/deluge-torrent/deluge/jobs/36309771

-from deluge.ui.client import Client, client
+from deluge.ui.client import client, Client
@cas--
Copy link
Author

cas-- commented Sep 26, 2014

I did a little debugging and think it's related to _module_key() setting the key to lower so that when it is passed to natsorted it will be based on the (random) order the modules were added to from_modules list.

It appears that the workaround for now is to use order-by-type.

cas-- added a commit to deluge-torrent/deluge that referenced this issue Sep 26, 2014
@timothycrosley
Copy link
Member

Hi @cas--,

Thanks for reporting this issue, I plan on making order_by_type the default behavior in the next release which should fix this issue.

Thanks!

~Timothy Crosley

@timothycrosley
Copy link
Member

A new version (3.9.1) has been released that fixes this issue.

Thanks again for reporting!

~Timothy

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

No branches or pull requests

2 participants