remove TODOs

Author: lukaszsamson

apps/language_server/lib/language_server/providers/completion.ex

@@ -104,7 +104,6 @@ defmodule ElixirLS.LanguageServer.Providers.Completion do
 
     prefix = get_prefix(text_before_cursor)
 
-    # TODO: Don't call into here directly
     # Can we use ElixirSense.Providers.Suggestion? ElixirSense.suggestions/3
     metadata = ElixirSense.Core.Parser.parse_string(text, true, true, line)
 

apps/language_server/lib/language_server/providers/workspace_symbols.ex

@@ -385,7 +385,6 @@ defmodule ElixirLS.LanguageServer.Providers.WorkspaceSymbols do
       chunked_module_paths
       |> do_process_chunked(fn chunk ->
         for {module, path} <- chunk,
-            # TODO: Don't call into here directly
             {kind, {type, type_ast, args}} <-
               ElixirSense.Core.Normalized.Typespec.get_types(module),
             kind in [:type, :opaque] do
@@ -405,7 +404,6 @@ defmodule ElixirLS.LanguageServer.Providers.WorkspaceSymbols do
       |> do_process_chunked(fn chunk ->
         for {module, path} <- chunk,
             function_exported?(module, :behaviour_info, 1),
-            # TODO: Don't call into here directly
             {{callback, arity}, [{:type, location, _, _}]} <-
               ElixirSense.Core.Normalized.Typespec.get_callbacks(module) do
           {callback, arity} = SourceFile.strip_macro_prefix({callback, arity})

apps/language_server/lib/language_server/source_file.ex

@@ -154,7 +154,6 @@ defmodule ElixirLS.LanguageServer.SourceFile do
   end
 
   def module_line(module) do
-    # TODO: Don't call into here directly
     case ElixirSense.Core.Normalized.Code.get_docs(module, :moduledoc) do
       nil ->
         nil
@@ -165,7 +164,6 @@ defmodule ElixirLS.LanguageServer.SourceFile do
   end
 
   def function_line(mod, fun, arity) do
-    # TODO: Don't call into here directly
     case ElixirSense.Core.Normalized.Code.get_docs(mod, :docs) do
       nil ->
         nil