Skip to content

Remove obsolete imports#63

Merged
math2001 merged 1 commit intomath2001:masterfrom
deathaxe:pr/obsolete-imports
Jan 19, 2020
Merged

Remove obsolete imports#63
math2001 merged 1 commit intomath2001:masterfrom
deathaxe:pr/obsolete-imports

Conversation

@deathaxe
Copy link
Copy Markdown
Contributor

Looks like from ..libs.sublimefunctions import * implicitly imports its imported modules as well. Hence import os.path in create.py can be removed.

Furthermore remove the empty line between the coding utf8 line and the first import in some files to gain consistent look of sources.

Looks like `from ..libs.sublimefunctions import *` implicitly imports
its imported modules as well. Hence `import os.path` in create.py can
be removed.

Furthermore remove the empty line between the `coding utf8` line and
the first import in some files to gain consistent look of sources.
@math2001 math2001 merged commit d463c82 into math2001:master Jan 19, 2020
@math2001
Copy link
Copy Markdown
Owner

Reorganizing the imports is something I want to do at some point, because I find it absolutely horrible to have os.path imported by some random from sublimefunctions import *. Plus it doesn't play nice with linting.

@deathaxe deathaxe deleted the pr/obsolete-imports branch April 12, 2021 19:31
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