Check in macros

lib/credo/check/consistency/unused_variable_names/collector.ex

@@ -22,7 +22,7 @@ defmodule Credo.Check.Consistency.UnusedVariableNames.Collector do
   end
 
   defp traverse(callback, {def, _, [{_, _, params} | _]} = ast, acc)
-       when def in [:def, :defp] do
+       when def in [:def, :defp, :defmacro, :defmacrop] do
     {ast, reduce_unused_variables(params, callback, acc)}
   end
 

test/credo/check/consistency/unused_variable_names_test.exs

@@ -286,6 +286,36 @@ defmodule Credo.Check.Consistency.UnusedVariableNamesTest do
     end)
   end
 
+  test "it should report a violation for different naming schemes with a macro (expects meaningful)" do
+    [
+      """
+      defmodule Credo.SampleOne do
+        defmodule Foo do
+          defmacro __using__(_) do
+          end
+        end
+
+        def bar(_opts) do
+        end
+      end
+      """,
+      """
+      defmodule Credo.SampleTwo do
+        defmodule Foo do
+          defmacrop bar(_opts) do
+          end
+        end
+      end
+      """
+    ]
+    |> to_source_files()
+    |> run_check(@described_check)
+    |> assert_issue(fn issue ->
+      assert "_" == issue.trigger
+      assert 3 == issue.line_no
+    end)
+  end
+
   test "it should report a violation for naming schemes other than the forced one" do
     [
       """