Adds trailing comma even when asked not to

[jml]

long.py

from memflow.operators.itunes_connect_reports_to_file_operator import (
    ITunesConnectReportsToFileOperator,
)

.isort.cfg:

[settings]
# This must match the line length for `black` in pyproject.toml.
line_length=100
# We also want imports in __init__.py sorted.
not_skip=__init__.py
# Two blank lines between the imports and the rest of the code.
# This is a subjective preference.
lines_after_imports=2

# These are necessary for `isort` to create import statements that are
# compatible with `black`. Changing these will break our auto-formatting.
# See <https://black.readthedocs.io/en/stable/the_black_code_style.html>.
multi_line_output=3
include_trailing_comma=True
force_grid_wrap=0
combine_as_imports=True

$ isort -rc
--- /Users/jml/Desktop/isort-bug/long.py:before	2019-02-25 12:27:39.166725
+++ /Users/jml/Desktop/isort-bug/long.py:after	2019-02-25 12:28:48.686402
@@ -1,3 +1,3 @@
 from memflow.operators.itunes_connect_reports_to_file_operator import (
-    ITunesConnectReportsToFileOperator,
+    ITunesConnectReportsToFileOperator
 )

I expected this to leave the file unchanged, preserving the trailing comma.

isort 4.3.4 works as expected. 4.3.5 through 4.3.8 all behave as above.

[tino]

Yup, having the same problem here.

[jaswilli]

It looks like the change came in here: 75b3549

And I'm not sure what Black was doing at the time of the above change but at least now it definitely seems to be inserting a trailing comma in this situation (python 3.7.2, black 18.9b0).

[timothycrosley]

We've done a hot-fix using @viourr's pull request, that fixes this issue.

Thank you!

~Timothy