change autocorrect to autofix

Author: devonremote

.DS_Store

@@ -154,7 +154,7 @@ defmodule Credo.Check do
               Credo.Issue.t()
 
   @doc false
-  @callback autocorrect(source_file :: String.t(), issue :: Issue.t()) :: String.t()
+  @callback autofix(source_file :: String.t(), issue :: Issue.t()) :: String.t()
 
   @base_category_exit_status_map %{
     consistency: 1,
@@ -400,7 +400,7 @@ defmodule Credo.Check do
 
       @doc false
       @impl true
-      def autocorrect(source_file, _issue) do
+      def autofix(source_file, _issue) do
         source_file
       end
 

lib/credo/check.ex

@@ -45,14 +45,14 @@ defmodule Credo.Check.Consistency.LineEndings do
     "File is using unix line endings while most of the files use windows line endings."
   end
 
-  def autocorrect(file, issue) do
+  def autofix(file, issue) do
     if issue.message == message_for(:windows) do
-      do_autocorrect(file, :unix_to_windows)
+      do_autofix(file, :unix_to_windows)
     else
-      do_autocorrect(file, :windows_to_unix)
+      do_autofix(file, :windows_to_unix)
     end
   end
 
-  defp do_autocorrect(file, :windows_to_unix), do: String.replace(file, "\r\n", "\n")
-  defp do_autocorrect(file, :unix_to_windows), do: String.replace(file, "\n", "\r\n")
+  defp do_autofix(file, :windows_to_unix), do: String.replace(file, "\r\n", "\n")
+  defp do_autofix(file, :unix_to_windows), do: String.replace(file, "\n", "\r\n")
 end

lib/credo/check/consistency/line_endings.ex

@@ -224,20 +224,20 @@ defmodule Credo.Check.Readability.AliasOrder do
     )
   end
 
-  def autocorrect(file, _issue) do
+  def autofix(file, _issue) do
     {:ok, quoted} = :"Elixir.Code".string_to_quoted(file)
 
     modified =
       quoted
-      |> Macro.prewalk(&do_autocorrect/1)
+      |> Macro.prewalk(&do_autofix/1)
       |> Macro.to_string()
       |> :"Elixir.Code".format_string!()
       |> to_string()
 
     "#{modified}\n"
   end
 
-  defp do_autocorrect({:__block__ = op, meta, [{:alias, _, _} | _] = aliases}) do
+  defp do_autofix({:__block__ = op, meta, [{:alias, _, _} | _] = aliases}) do
     modified =
       aliases
       |> group_aliases()
@@ -253,7 +253,7 @@ defmodule Credo.Check.Readability.AliasOrder do
     {op, Keyword.delete(meta, :line), modified}
   end
 
-  defp do_autocorrect(ast), do: ast
+  defp do_autofix(ast), do: ast
 
   defp put_line_number([{op, meta, args} | tail], line) do
     modified = {op, Keyword.put(meta, :line, line), args}

lib/credo/check/readability/alias_order.ex

@@ -40,7 +40,7 @@ defmodule Credo.Check.Readability.TrailingBlankLine do
     )
   end
 
-  def autocorrect(file, _issue) do
+  def autofix(file, _issue) do
     "#{file}\n"
   end
 end

lib/credo/check/readability/trailing_blank_line.ex

@@ -72,22 +72,22 @@ defmodule Credo.Check.Refactor.UnlessWithElse do
     )
   end
 
-  def autocorrect(file, _issue) do
+  def autofix(file, _issue) do
     {:ok, quoted} = :"Elixir.Code".string_to_quoted(file)
 
     modified =
       quoted
-      |> Macro.prewalk(&do_autocorrect/1)
+      |> Macro.prewalk(&do_autofix/1)
       |> Macro.to_string()
       |> :"Elixir.Code".format_string!()
       |> to_string()
 
     "#{modified}\n"
   end
 
-  defp do_autocorrect({:unless, meta, [clause, [do: falsy, else: truthy]]}) do
+  defp do_autofix({:unless, meta, [clause, [do: falsy, else: truthy]]}) do
     {:if, meta, [clause, [do: truthy, else: falsy]]}
   end
 
-  defp do_autocorrect(ast), do: ast
+  defp do_autofix(ast), do: ast
 end

lib/credo/check/refactor/unless_with_else.ex

@@ -46,7 +46,7 @@ defmodule Credo.Check.Warning.UnusedStringOperation do
     )
   end
 
-  def autocorrect(file, _issue) do
+  def autofix(file, _issue) do
     {_, quoted} = Credo.Code.ast(file)
     source_file = SourceFile.parse(file, "nofile")
 
@@ -60,15 +60,15 @@ defmodule Credo.Check.Warning.UnusedStringOperation do
 
     modified =
       quoted
-      |> Macro.prewalk(&do_autocorrect(&1, unused_calls))
+      |> Macro.prewalk(&do_autofix(&1, unused_calls))
       |> Macro.to_string()
       |> :"Elixir.Code".format_string!()
       |> to_string()
 
     "#{modified}\n"
   end
 
-  defp do_autocorrect({:__block__, meta, [{:|>, _pipe_meta, pipe_args} | tail] = args}, unused_calls) do
+  defp do_autofix({:__block__, meta, [{:|>, _pipe_meta, pipe_args} | tail] = args}, unused_calls) do
     args =
       if List.last(pipe_args) in unused_calls do
         [hd(pipe_args) | tail]
@@ -79,13 +79,13 @@ defmodule Credo.Check.Warning.UnusedStringOperation do
     {:__block__, meta, args}
   end
 
-  defp do_autocorrect({:__block__, meta, args}, unused_calls) do
+  defp do_autofix({:__block__, meta, args}, unused_calls) do
     {:__block__, meta, Enum.reject(args, & &1 in unused_calls)}
   end
 
-  defp do_autocorrect({:do, node}, unused_calls) do
-    {:do, do_autocorrect(node, unused_calls)}
+  defp do_autofix({:do, node}, unused_calls) do
+    {:do, do_autofix(node, unused_calls)}
   end
 
-  defp do_autocorrect(ast, _), do: ast
+  defp do_autofix(ast, _), do: ast
 end

lib/credo/check/warning/unused_string_operation.ex

@@ -30,7 +30,7 @@ defmodule Credo.CLI.Command.Suggest.SuggestCommand do
       Switch.boolean("read_from_stdin"),
       Switch.boolean("strict"),
       Switch.boolean("verbose"),
-      Switch.boolean("autocorrect"),
+      Switch.boolean("autofix"),
       Switch.boolean("watch")
     ]
 
@@ -42,7 +42,7 @@ defmodule Credo.CLI.Command.Suggest.SuggestCommand do
       print_before_analysis: [__MODULE__.PrintBeforeInfo],
       run_analysis: [Task.RunChecks],
       filter_issues: [Task.SetRelevantIssues],
-      run_autocorrect: [Task.RunAutocorrect],
+      run_autofix: [Task.RunAutofix],
       print_after_analysis: [__MODULE__.PrintResultsAndSummary]
     )
   end

lib/credo/cli/command/suggest/suggest_command.ex

@@ -1,18 +1,18 @@
-defmodule Credo.CLI.Task.RunAutocorrect do
+defmodule Credo.CLI.Task.RunAutofix do
   @moduledoc false
 
   use Credo.Execution.Task
 
   def call(exec, opts, read_fun \\ &File.read!/1, write_fun \\ &File.write!/2) do
-    if exec.autocorrect do
+    if exec.autofix do
       issues = Keyword.get_lazy(opts, :issues, fn -> Execution.get_issues(exec) end)
 
       issues
       |> group_by_file
       |> Enum.each(fn {file_path, issues} ->
         file = read_fun.(file_path)
 
-        corrected = Enum.reduce(issues, file, &run_autocorrect(&1, &2, exec))
+        corrected = Enum.reduce(issues, file, &run_autofix(&1, &2, exec))
 
         write_fun.(file_path, corrected)
       end)
@@ -27,8 +27,8 @@ defmodule Credo.CLI.Task.RunAutocorrect do
     end)
   end
 
-  defp run_autocorrect(issue, file, exec) do
-    case issue.check.autocorrect(file, issue) do
+  defp run_autofix(issue, file, exec) do
+    case issue.check.autofix(file, issue) do
       ^file ->
         file
 

lib/credo/cli/task/run_autocorrect.ex

@@ -81,7 +81,7 @@ defmodule Credo.ConfigBuilder do
     |> add_switch_ignore(switches)
     |> add_switch_mute_exit_status(switches)
     |> add_switch_only(switches)
-    |> add_switch_autocorrect(switches)
+    |> add_switch_autofix(switches)
     |> add_switch_read_from_stdin(switches)
     |> add_switch_strict(switches)
     |> add_switch_min_priority(switches)
@@ -263,13 +263,13 @@ defmodule Credo.ConfigBuilder do
 
   defp add_switch_only(exec, _), do: exec
 
-  # add_switch_autocorrect
+  # add_switch_autofix
 
-  defp add_switch_autocorrect(exec, %{autocorrect: true}) do
-    %Execution{exec | autocorrect: true}
+  defp add_switch_autofix(exec, %{autofix: true}) do
+    %Execution{exec | autofix: true}
   end
 
-  defp add_switch_autocorrect(exec, _), do: exec
+  defp add_switch_autofix(exec, _), do: exec
 
   # add_switch_ignore