Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Duplicate suggestions #1175

Open
eksperimental opened this issue Jan 9, 2025 · 0 comments · May be fixed by #1178
Open

Duplicate suggestions #1175

eksperimental opened this issue Jan 9, 2025 · 0 comments · May be fixed by #1178

Comments

@eksperimental
Copy link

Precheck

Here a repo that reproduces the issue:
https://github.com/eksperimental-debug/credo-issues

Environment

  • Credo version (mix credo -v): 1.7.11
  • Erlang/Elixir version (elixir -v): Elixir 1.18.1 (compiled with Erlang/OTP 27)

What were you trying to do?

mix credo --script

Expected outcome

To have only one suggestion

0### Actual outcome

❯ mix credo --strict

  Consistency
┃
┃ [C] ↗ Unused variables should be named consistently. It seems your strategy is to give them meaningful names (eg. `_foo`) but `_` does not follow that convention.
┃       lib/foo_web/components/core_components.ex:4:31 #(FooWeb.CoreComponents.icon)
┃ [C] ↗ Unused variables should be named consistently. It seems your strategy is to give them meaningful names (eg. `_foo`) but `_` does not follow that convention.
┃       lib/foo_web/components/core_components.ex:4:31 #(FooWeb.CoreComponents.icon)
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix duplicated suggestions with UnusedVariableNamesTest quangngd/credo
1 participant
@eksperimental