elvis_core
elvis_core copied to clipboard
inaka
The core of an Erlang linter
## Bug Description The `no_successive_maps` rule (see #157 and #255) doesn't warn when using nested maps. ## To Reproduce This triggers the warning: ``` _ = [ #{a => b,...
## Bug Description **Note**: might not be a bug. `max_function_length` complains if a function definition spans over a given number of lines (by default: 30)... ... **but** it's also taking...
### Name `consistent_deps_applications` ### Brief Description It's common to have `.app.src` updated with the dep names from `rebar.config`. This rule would look at `rebar.config`'s `deps`, and compare them with the...
Think about removing `no_successive_maps` when https://github.com/erlang/otp/pull/8069 is a reality for at least 3 versions, or less, depending on how "aggressive" we wanna be.
### Name # No Init Lists ### Brief Description The idea is to avoid code like this one… ```erlang -module my_server. % …exports and everything else… start_link(AParam) -> gen_server:start_link(?MODULE, [AParam],...
This issue might evolve into a new rule proposal, after which I'll update the description accordingly. If https://github.com/erlang/otp/pull/7407 goes forward, do you think Elvis would have a say on (what...
Should Elvis care that a **consecutive string** is being used, as it does with maps? e.g. `"a " "long " "string!"`, but only if found in the same line? (I'm...
### Name `unsafe_application_get` ### Brief Description In the context of an application, while it's technically possible to do `application:get_env(_)` for another application, it's usually a bad idea. ### Reasoning Very...
### Name `no_import_type` ### Brief Description It is possible that a new `import_type` directive (⚠️ a previous link was removed - though GitHub doesn't seem to update the target -...
### Name None yet. ### Brief Description Should we propose some of the base functions (that have a timeout counterpart) to be added to the list of `no_common_caveats` functions? We...