Remove migrations indirection

Author: crbelaus

dev.exs

@@ -145,8 +145,8 @@ end
 defmodule Migration0 do
   use Ecto.Migration
 
-  def up, do: ErrorTracker.Migrations.up(prefix: "private")
-  def down, do: ErrorTracker.Migrations.down(prefix: "private")
+  def up, do: ErrorTracker.Migration.up(prefix: "private")
+  def down, do: ErrorTracker.Migration.down(prefix: "private")
 end
 
 Application.put_env(:phoenix, :serve_endpoints, true)

lib/error_tracker/migration.ex

@@ -1,10 +1,3 @@
-defmodule ErrorTracker.Migrations do
-  @moduledoc false
-
-  defdelegate up(opts \\ []), to: ErrorTracker.Migration
-  defdelegate down(opts \\ []), to: ErrorTracker.Migration
-end
-
 defmodule ErrorTracker.Migration do
   @moduledoc """
   Create and modify the database tables for the ErrorTracker.
@@ -25,8 +18,8 @@ defmodule ErrorTracker.Migration do
   defmodule MyApp.Repo.Migrations.AddErrorTracker do
     use Ecto.Migration
 
-    def up, do: ErrorTracker.Migrations.up()
-    def down, do: ErrorTracker.Migrations.down()
+    def up, do: ErrorTracker.Migration.up()
+    def down, do: ErrorTracker.Migration.down()
   end
   ```
 
@@ -51,8 +44,8 @@ defmodule ErrorTracker.Migration do
   defmodule MyApp.Repo.Migrations.UpdateErrorTrackerToVN do
     use Ecto.Migration
 
-    def up, do: ErrorTracker.Migrations.up(version: N)
-    def down, do: ErrorTracker.Migrations.down(version: N)
+    def up, do: ErrorTracker.Migration.up(version: N)
+    def down, do: ErrorTracker.Migration.down(version: N)
   end
   ```
 
@@ -74,8 +67,8 @@ defmodule ErrorTracker.Migration do
   defmodule MyApp.Repo.Migrations.AddErrorTracker do
     use Ecto.Migration
 
-    def up, do: ErrorTracker.Migrations.up(prefix: "custom_schema")
-    def down, do: ErrorTracker.Migrations.down(prefix: "custom_schema")
+    def up, do: ErrorTracker.Migration.up(prefix: "custom_schema")
+    def down, do: ErrorTracker.Migration.down(prefix: "custom_schema")
   end
   ```
 
@@ -87,8 +80,8 @@ defmodule ErrorTracker.Migration do
   defmodule MyApp.Repo.Migrations.AddErrorTracker do
     use Ecto.Migration
 
-    def up, do: ErrorTracker.Migrations.up(prefix: "custom_schema", create_schema: false)
-    def down, do: ErrorTracker.Migrations.down(prefix: "custom_schema")
+    def up, do: ErrorTracker.Migration.up(prefix: "custom_schema", create_schema: false)
+    def down, do: ErrorTracker.Migration.down(prefix: "custom_schema")
   end
   ```
 
@@ -118,7 +111,7 @@ defmodule ErrorTracker.Migration do
 
   defp migrator do
     case ErrorTracker.Repo.__adapter__() do
-      Ecto.Adapters.Postgres -> ErrorTracker.Migrations.Postgres
+      Ecto.Adapters.Postgres -> ErrorTracker.Migration.Postgres
       adapter -> raise "ErrorTracker does not support #{adapter}"
     end
   end

lib/error_tracker/migrations/postgres.ex renamed to lib/error_tracker/migration/postgres.ex

@@ -1,4 +1,4 @@
-defmodule ErrorTracker.Migrations.Postgres do
+defmodule ErrorTracker.Migration.Postgres do
   @moduledoc false
 
   @behaviour ErrorTracker.Migration

lib/error_tracker/migrations/postgres/v01.ex renamed to lib/error_tracker/migration/postgres/v01.ex

@@ -1,4 +1,4 @@
-defmodule ErrorTracker.Migrations.Postgres.V01 do
+defmodule ErrorTracker.Migration.Postgres.V01 do
   @moduledoc false
 
   use Ecto.Migration

live.exs

@@ -1,5 +1,7 @@
 Mix.install([
-  {:phoenix_playground, github: "phoenix-playground/phoenix_playground", ref: "ee6da0fc3b141f78b9f967ce71a4fb015c6764a6"},
+  {:phoenix_playground,
+   github: "phoenix-playground/phoenix_playground",
+   ref: "ee6da0fc3b141f78b9f967ce71a4fb015c6764a6"},
   {:error_tracker, path: "."}
 ])
 
@@ -11,18 +13,19 @@ end
 Application.put_env(:error_tracker, :repo, ErrorTrackerDev.Repo)
 Application.put_env(:error_tracker, :application, :error_tracker_dev)
 Application.put_env(:error_tracker, :prefix, "private")
-Application.put_env(:error_tracker, ErrorTrackerDev.Repo, url: "ecto://postgres:postgres@127.0.0.1/error_tracker_dev")
 
+Application.put_env(:error_tracker, ErrorTrackerDev.Repo,
+  url: "ecto://postgres:postgres@127.0.0.1/error_tracker_dev"
+)
 
 # This migration will set up the database structure
 defmodule Migration0 do
   use Ecto.Migration
 
-  def up, do: ErrorTracker.Migrations.up(prefix: "private")
-  def down, do: ErrorTracker.Migrations.down(prefix: "private")
+  def up, do: ErrorTracker.Migration.up(prefix: "private")
+  def down, do: ErrorTracker.Migration.down(prefix: "private")
 end
 
-
 defmodule DemoLive do
   use Phoenix.LiveView
 
@@ -54,7 +57,6 @@ defmodule DemoLive do
     """
   end
 
-
   def handle_event("inc", _params, socket) do
     {:noreply, assign(socket, count: socket.assigns.count + 1)}
   end
@@ -71,15 +73,13 @@ defmodule DemoLive do
     if params["crash"] == "handle_params" do
       raise "Crash on handle_params"
     end
+
     {:noreply, socket}
   end
 end
 
-
-
 PhoenixPlayground.start(live: DemoLive, child_specs: [ErrorTrackerDev.Repo])
 
-
 # Create the database if it does not exist and run migrations if needed
 _ = Ecto.Adapters.Postgres.storage_up(ErrorTrackerDev.Repo.config())