ci: configure isort for code formatting

[williballenthin]

$ isort --length-sort --line-width 120 --thirdparty idc --thirdparty idaapi --thirdparty idautils --thirdparty ida_gdl --thirdparty PyQt5 --thirdparty argparse --builtin posixpath --thirdparty tabulate --thirdparty viv_utils --recursive .

[williballenthin]

run this before black, due to some line width differences.

[Ana06]

I think we should use isort5. It includes a black profile (see PyCQA/isort#694)

isort --profile black --length-sort --line-width 120 .
black -l 120 .

Running it this way black doesn't change things back.

Note that isort5 has removed the recursive option (which is the default now) and changed the module placement which affects the thirdparty option. So I am not sure if we need/want to add more options to sort. @williballenthin please let me know if more options are needed, so that I add this to the ci.

[williballenthin]

can you summarize how our code style will change by using these different options? maybe include a snippet or two that shows the changes.

given that we always run isort and then black, i'm not too concerned if there are minor incompatibilities. but, if there's a recommended way of keeping things in sync, and it matches our code style, that's definitely best.

[Ana06]

You can see the changes here: f4e6541

Some changes seems related to argparse not being a third party library. The rest seems to be improvements in the new version.

[williballenthin]

the changes look very reasonable. i'm on board.