Fix false positive for NestedFunctionCalls

Refs #1170

Author: rrrene

lib/credo/check/readability/nested_function_calls.ex

@@ -68,6 +68,11 @@ defmodule Credo.Check.Readability.NestedFunctionCalls do
     {nil, acc}
   end
 
+  # We don't look into @type attributes
+  defp traverse({:@, _, [{:type, _, _args}]}, acc, _issue) do
+    {nil, acc}
+  end
+
   # Any call
   defp traverse(
          {{_name, _loc, call}, meta, args} = ast,
@@ -137,7 +142,7 @@ defmodule Credo.Check.Readability.NestedFunctionCalls do
       :"<|>",
       :"^^^",
       :"~~~",
-      :"..//"
+      :..//
     ]
 
     def cannot_be_in_pipeline?(

test/credo/check/readability/nested_function_calls_test.exs

@@ -10,7 +10,8 @@ defmodule Credo.Check.Readability.NestedFunctionCallsTest do
   test "it should NOT report code with no nested function calls" do
     """
     defmodule CredoSampleModule do
-      def some_code do
+      @type type_name :: Keyword.t(Some.remote(some_arg))
+      def some_code(_foo) do
         Enum.shuffle([1,2,3])
       end
     end