credo
credo copied to clipboard
rrrene
A static code analysis tool for the Elixir language with a focus on code consistency and teaching.
I had false positives on this when I had changed from `apply(module, fun, [arg1, arg2])` to a piped version (because there was another call around it). I decided to make...
Hey 👋 First of all thanks for all the work and effort you put in Credo 🙇 ### Precheck * For bugs, please do a quick search and make sure...
I'm doing a bit of preliminary testing with https://github.com/elixir-lang/elixir/releases/tag/v1.14.0-rc.1, and noticed the following warning, so I'm opening an issue to report it: ``` ==> credo Compiling 242 files (.ex) warning:...
### 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...
This is a check I have wanted myself for a while. I thought I would submit it upstream in case it's useful to others. I have added it as disabled...
### Environment * Credo version (`mix credo -v`): 1.7.0 * Erlang/Elixir version (`elixir -v`): checked multiple versions * Operating system: MacOS ### What were you trying to do? I was...
### Precheck * For bugs, please do a quick search and make sure the bug has not yet been reported here - Nothing found! ### Environment * Credo version (`mix...
### Precheck - [X] For bugs, please do a quick search and make sure the bug has not yet been reported here ### Environment * Credo version (`mix credo -v`):...
**I am re-opening this PR as I'm uncertain my comment on the previously closed PR were seen.** Specifically, see https://github.com/rrrene/credo/pull/935#issuecomment-1466309929 This PR adds a check named `Credo.Check.Readability.CondCatchallTrue` that checks for...
Previously, this was backfilling the column based on analyzing the source code line, but the column is already available in the AST, so we instead take it from there. This...
