credo

credo copied to clipboard

Allowing to ignore modules on Credo.Check.Readability.AliasAs

4

Closes https://github.com/rrrene/credo-proposals/issues/81 This PR is introducing a new `:ignore` param on `Credo.Check.Readability.AliasAs`, to allow users to configure a list of modules that should be ignored by this check. For example:...

kelvinst

[956] Add test and maybe a fix for modules passed as arguments

1

Related to #956 I want to check in early to make sure I'm on the right path. When a module is passed as an argument to a function, it should...

tpitale

Initial spike on Autocorrect

8

This PR is for #944. It's really just a spike at this point, but it works for the single check I've implemented autocorrect for! At the moment the UI is...

devonestes

Credo.Check.Consistency.ExceptionNames complains for two exceptions

3

### Precheck * Proposals for new features should be submitted via: https://github.com/rrrene/credo-proposals * For bugs, please do a quick search and make sure the bug has not yet been reported...

LostKobrakai

Slash inside macro

4

### Environment ``` 1.6.4-ref.deletion.8140e31+uncommittedchanges ``` ### What were you trying to do? ```elixir x = 1 p = path(x / x) ``` where `path` is a macro ### Expected outcome...

hissssst

add new check for pipe per line

4

Some style guide proposes: ```elixir # not preferred 1 |> Integer.to_string() |> String.to_integer() # preferred 1 |> Integer.to_string() |> String.to_integer() ``` This PR provides a check for that. -- Credits...

rands0n

credo wants a spec for a function that is defoverridable

2

### Environment * Credo version (`mix credo -v`): 1.6.4 * Erlang/Elixir version (`elixir -v`): Elixir 1.13.4 (compiled with Erlang/OTP 24) * Operating system: macOS 12.13 ### What were you trying...

lukad

Diff is checking files completely

4

### Environment * Credo version (`mix credo -v`): 1.6.1 * Erlang/Elixir version (`elixir -v`): Erlang/OTP 23 - Elixir 1.11.2 (compiled with Erlang/OTP 21) * Operating system: macOS Monterey ### What...

odarriba

Credo.Check.Design.AliasUsage doesn't warn when unaliased modules are used without function calls

2

Hi @rrrene! First off, thanks for Credo! ### Precheck Current master/main. ### Expected outcome Warn when modules are not aliased. More details below. ### Actual outcome It doesn't warn. ###...

pdgonzalez872

Create CondFinalCondition readability check

10

This PR adds a check named `Credo.Check.Readability.CondCatchallTrue` that checks for `cond`s that end in a catchall clause that use a literal value other than `true`. I created this check to...

Stratus3D