New rule: No Import

[elbrujohalcon]

No Import

Brief Description

The idea is to ban the usage of import attributes.

-module bad.
-import io, [format/1].
-export [f/0].

f() -> format("BAD BAD BAD!\n").

-module good.
-export [f/0].

f() -> io:format("GOOD GOOD GOOD!\n").

Should be on by default?

YES

Options

• None.

Reasoning

Writing fully-qualified names for functions is not that much of an extra effort and the resulting code is easier to comprehend (since readers will be immediately able to tell if a function is defined in the same module or not) and debug (since copy & pasting to try stuff on the console is easier).