René Föhring comments

Results 126 comments of


                                            René Föhring

Jump between test file and actual file not working im module name also contains "Test"

Following up: Any news on this? 🙂

Arguable advice from `Credo.Check.Refactor.AppendSingleItem`

While I think that I am getting what you are trying to say, I am not sure what you suggest to do about it. I understand why you are saying...

Predicate function names should not start with 'is', and should end in a question mark flagged for :ets.whereis

This sounds strange. Can you provide a complete code sample that triggers this?

Add Warning Check to Evaluate Total Field amount in Structs

Hi, I like the idea behind this check. Will try to review as soon as I can 👍

Incorrect `Exception modules should be named consistently`

Thx for reporting this! We should definitely fix this. It just has to be fixed for all consistency checks at once, because I decided a long time ago to create...

Credo.Check.Readability.FunctionNames potential false positives for non-sigil functions beginning with "sigil_"

Yeah, it seems like this issue and the referenced issue are at odds with each other 😰 Any suggestions how we could compromise here?

Credo.Check.Readability.FunctionNames potential false positives for non-sigil functions beginning with "sigil_"

Any new input on this?

RedundantWithClauseResult false positive when containing a map match

Thx for reporting! :+1:

AbcSize does not exclude method declarations.

Hi, thx for creating an issue. Could you please fill out the issue template you deleted and provide a code example that reproduces your issue?

Detect ambiguity in function calls without parentheses

The fact that the filename is `nofilename` on this is definitely bad and we should fix that 👍 (this is completely Credo's fault, I believe). Could you provide an Elixir...