Autoimport breaks positional elements (module docstring, shebang, file variables, ...) about autoimport HOT 4 CLOSED

Hi @kbauer, thanks for opening the issue,

I've solved it in #71 and released under version 0.6.0.

Cheers

from autoimport.

@lyz-code That was fast :)

However the fix might make too many assumptions. E.g.:

#!/usr/bin/env python3
# -*- coding: latin-1 -*-

# pylint: disable=foobar

"""

This is the docstring.

"""

import sys

print(os.path.exists(sys.argv[1]))

becomes

#!/usr/bin/env python3
# -*- coding: latin-1 -*-

import sys
import os

# pylint: disable=foobar

"""

This is the docstring.

"""


print(os.path.exists(sys.argv[1]))

Diff:

Apparently, autoimport also removes the trailing newline, which git isn't happy about.

from autoimport.

Hi, I see your points, I'll fix them tomorrow.

Regarding the trailing newline, I did it to be compliant with the pre-commit hook, but I understand that, it's something that should be handled by other tools, so I'll respect trailing newlines at the end of the file.

Thanks!

from autoimport.

@kbauer Fixed in #72 and released under 0.6.1. If you find any other case, do not hesitate to reopen it

from autoimport.