René Föhring

Results 126 comments of René Föhring

In my mind, we should not assume too much about the data we're given (especially when reading the source code of `String.replace_invalid` I am reminded of how much I do...

Why would we overwrite a message instead of just printing a warning? > I do think it's nice when a message, when searched, yields exactly the right results, so I...

This is part of Credo [v1.7.0-rc.1](https://hex.pm/packages/credo/1.7.0-rc.1) :+1:

This is part of Credo [v1.7.0-rc.1](https://hex.pm/packages/credo/1.7.0-rc.1) :+1:

@ding-an-sich Thx! :+1:

This is part of Credo [v1.7.0-rc.1](https://hex.pm/packages/credo/1.7.0-rc.1) :+1:

Thx for reporting! :+1: This is definitely an issue that we should solve!

@jjcarstens Thanks for reporting this 😀 It is now fixed on `master`. You can try this by setting the Credo dep to ```elixir {:credo, github: "rrrene/credo"} ``` Please report back...

@dkuku This does not look like an error caused by Credo. Since the name of the check module (`CredoChecks.NoAmbiguousAliases`) starts with `CredoChecks`, I would guess that the failure comes from...

And the reason why this is happening is probably that the AST representation for an `alias` call is ```elixir {:alias, meta, [_|_] = args} ``` where `args` represents the parameters...