René Föhring

As mentioned, this is great. Please take this critique with a grain of salt and as a compliment for putting this together. I would love for Credo 1.7 to provide...

I think we are not (yet) getting anywhere in this conversation. :-( I've written three different answers over the course of a week and none of them seem to fit...

This looks good. The last thing I am wondering is if we covered all edge cases in the tests though. Are there things we could be missing? Like "exotic" combinations...

Why do you think this should not raise an issue? Your code `x / x` is (on the surface) always resulting in the same result which Credo points out, unless...

You are right. So we will habe to find a way to statically derermine whether or not something is a function or a macro 👍

Sorry for not checking in for a while. So does that mean that we just have to insert a BOM at the start of the output for PowerShell to be...

@TheFirstAvenger I was able to install Erlang/Elixir/Credo on a Windows 10 box and got this in Powershell:  When I added a BOM in front of the output, it just...

You're right, this is a bug. But we can not solve it this way. :-( > [...] alters the prewalk check to unpipe those chains and insert > a phantom...

You're onto something here, but it is still too complex. The logic needs to be safe, and the first step towards safety is simplicity. We have two different use cases:...

Did you skip the `get_forbidden_pipe/2` function head for matching `Jason.decode/2` on purpose?