Use error, improve docs

Author: josevalim

lib/mix/lib/mix/dep/converger.ex

@@ -381,7 +381,7 @@ defmodule Mix.Dep.Converger do
   end
 
   defp warn_uneeded_override(dep, parent) do
-    Mix.shell().info(
+    Mix.shell().error(
       "Dependency #{Mix.Dep.format_dep(dep)} no longer requires :override on #{inspect(parent)}"
     )
   end

lib/mix/lib/mix/tasks/deps.ex

@@ -99,11 +99,11 @@ defmodule Mix.Tasks.Deps do
       targets (like `[:host, :rpi3]`)
 
     * `:override` - if set to `true` the dependency will override any other
-      definitions of itself by other dependencies. This option can also be a
-      list, which overrides the definitions of only certain dependencies. If
-      a dependency is not included in the list and an override is required,
-      it will still fail. If a dependency is in the list and no longer necessary,
-      then a warning is emitted
+      definitions of itself by other dependencies. From Elixir v1.20.0,
+      this option can also be a list of dependency names, which allows you to
+      override the definition of specific dependencies. If a dependency is not
+      included in the list and an override is required, it will still fail.
+      If a dependency is in the list and no longer necessary, then an error is emitted
 
     * `:manager` - Mix can also compile Rebar3 and makefile projects
       and can fetch sub dependencies of Rebar3 projects. Mix will

lib/mix/test/mix/dep_test.exs

@@ -1280,7 +1280,7 @@ defmodule Mix.DepTest do
       end)
 
       message = "Dependency ok 0.1.0 (deps/ok) no longer requires :override on :deps_repo"
-      assert_received {:mix_shell, :info, [^message]}
+      assert_received {:mix_shell, :error, [^message]}
     end
 
     test "warns when overrides are no longer needed" do
@@ -1298,7 +1298,7 @@ defmodule Mix.DepTest do
       message =
         "Dependency git_repo (#{fixture_path("git_repo")}) no longer requires :override on :deps_repo"
 
-      assert_received {:mix_shell, :info, [^message]}
+      assert_received {:mix_shell, :error, [^message]}
     end
 
     test "targets divergence when app is not overridden" do