feat: add git option

lib/mix/tasks/edit/add.ex

@@ -13,6 +13,7 @@ defmodule Mix.Tasks.Edit.Add do
   OPTS:
     --version         Set the version for the DEP
     --path            Set the path for the DEP
+    --git             Set the git repo for the DEP
     --only            Setting the only flag, example 'test' or 'test+dev'
     --override        Set the override option
     --no-runtime      Set the runtime option to false

lib/mix/tasks/edit/update.ex

@@ -11,6 +11,7 @@ defmodule Mix.Tasks.Edit.Update do
   OPTS:
     --version         Set the version for the DEP
     --path            Set the path for the DEP
+    --git             Set the git repo for the DEP
     --only            Setting the only flag, example 'test' or 'test+dev'
     --override        Set the override option
     --no-runtime      Set the runtime option to false

lib/mix_edit.ex

@@ -233,6 +233,11 @@ defmodule MixEdit do
         [
           {{:__block__, [format: :keyword], [:path]}, {:__block__, [delimiter: "\""], [value]}}
         ]
+
+      {:git, value} ->
+        [
+          {{:__block__, [format: :keyword], [:git]}, {:__block__, [delimiter: "\""], [value]}}
+        ]
     end)
   end
 

lib/mix_edit/general_task.ex

@@ -5,6 +5,7 @@ defmodule MixEdit.GeneralTask do
 
   @options [
     version: :string,
+    git: :string,
     sorted: :boolean,
     out: :string,
     in: :string,
@@ -98,7 +99,7 @@ defmodule MixEdit.GeneralTask do
       |> Enum.flat_map(fn {:remove, _, removed} -> removed end)
       |> Enum.map(&to_string/1)
 
-    unless removed == [] do
+    if removed != [] do
       Mix.Task.run("deps.unlock", removed)
       Mix.Task.run("deps.clean", removed)
     end
@@ -192,6 +193,7 @@ defmodule MixEdit.GeneralTask do
     )
   end
 
+  defp version_or_path(%{git: git}), do: "git `#{git}`"
   defp version_or_path(%{version: version}), do: "version `#{version}`"
   defp version_or_path(%{path: path}), do: "path `#{path}`"
 

test/mix_add_test.exs

@@ -203,5 +203,58 @@ defmodule MixEditTest do
     end
   end
 
+  test "sorting the dependency list works" do
+    deps = """
+    defmodule Test.MixProject do
+      use Mix.Project
+
+      def project do
+        [
+          version: "0.1.0",
+          start_permanent: Mix.env() == :prod,
+          deps: deps()
+        ]
+      end
+
+      defp deps do
+        [
+          {:jason, "~> 0.0.0"},
+          {:ex_doc, ">= 0.0.0"},
+          {:sweet_xml, ">= 0.0.0"}
+        ]
+      end
+    end
+    """
+
+    opts = %{
+      dependencies: [testing: [version: ">= 0.0.0"]],
+      sorted: true,
+      app: :test,
+      method: :add
+    }
+
+    {quoted, comments} = MixEdit.quote_string!(deps)
+
+    {new_file, info} =
+      Macro.prewalk(
+        quoted,
+        nil,
+        &MixEdit.deps_walker(&1, &2, opts)
+      )
+
+    assert {:add, [testing: [version: ">= 0.0.0"]]} == info
+
+    stringified =
+      new_file
+      |> Code.Formatter.to_algebra(comments: comments)
+      |> Inspect.Algebra.format(:infinity)
+      |> IO.iodata_to_binary()
+      |> String.replace("\n", " ")
+      |> String.replace(~r/\s+/, " ")
+
+    assert stringified =~
+             "defp deps do [{:ex_doc, \">= 0.0.0\"}, {:jason, \"~> 0.0.0\"}, {:sweet_xml, \">= 0.0.0\"}, {:testing, \">= 0.0.0\"}] end"
+  end
+
   defp unwrap_deps({{:__block__, _, [deps]}, _}), do: deps
 end