PR to track downstream changes in a fork

.formatter.exs

@@ -0,0 +1,5 @@
+# Used by "mix format"
+[
+  line_length: 110,
+  inputs: ["mix.exs", "{config,lib,test}/**/*.{ex,exs}"]
+]

.gitignore

@@ -34,6 +34,7 @@ GitHub.sublime-settings
 .tags_sorted_by_file
 .vagrant
 .DS_Store
+.elixir_ls
 
 # Ignore released binaries
 .deliver

config/.credo.exs

@@ -8,7 +8,7 @@
       checks: [
         {Credo.Check.Design.TagTODO, exit_status: 0},
         {Credo.Check.Readability.MaxLineLength, priority: :low, max_length: 120},
-        {Credo.Check.Readability.Specs, exit_status: 0},
+        {Credo.Check.Readability.Specs, exit_status: 0}
       ]
     }
   ]

config/dogma.exs

@@ -4,10 +4,10 @@ alias Dogma.Rule
 config :dogma,
   rule_set: Dogma.RuleSet.All,
   exclude: [
-    ~r(\Adeps/),
+    ~r(\Adeps/)
   ],
   override: [
-    %Rule.LineLength{ max_length: 120 },
-    %Rule.InfixOperatorPadding{ enabled: false },
-    %Rule.FunctionArity{ max: 5 },
+    %Rule.LineLength{max_length: 120},
+    %Rule.InfixOperatorPadding{enabled: false},
+    %Rule.FunctionArity{max: 5}
   ]

lib/ecto_trail/changelog.ex

@@ -4,13 +4,12 @@ defmodule EctoTrail.Changelog do
   """
   use Ecto.Schema
 
-  @primary_key {:id, :binary_id, autogenerate: true}
   schema Application.fetch_env!(:ecto_trail, :table_name) do
-    field :actor_id, :string
-    field :resource, :string
-    field :resource_id, :string
-    field :changeset, :map
+    field(:actor_id, :string)
+    field(:resource, :string)
+    field(:resource_id, :string)
+    field(:changeset, :map)
 
-    timestamps([type: :utc_datetime, updated_at: false])
+    timestamps(type: :utc_datetime, updated_at: false)
   end
 end

lib/ecto_trail/ecto_trail.ex

@@ -56,10 +56,11 @@ defmodule EctoTrail do
 
       Insert arguments, return and options same as `c:Ecto.Repo.insert/2` has.
       """
-      @spec insert_and_log(struct_or_changeset :: Ecto.Schema.t | Ecto.Changeset.t,
-                           actor_id :: String.T,
-                           opts :: Keyword.t) ::
-            {:ok, Ecto.Schema.t} | {:error, Ecto.Changeset.t}
+      @spec insert_and_log(
+              struct_or_changeset :: Ecto.Schema.t() | Ecto.Changeset.t(),
+              actor_id :: String.T,
+              opts :: Keyword.t()
+            ) :: {:ok, Ecto.Schema.t()} | {:error, Ecto.Changeset.t()}
       def insert_and_log(struct_or_changeset, actor_id, opts \\ []),
         do: EctoTrail.insert_and_log(__MODULE__, struct_or_changeset, actor_id, opts)
 
@@ -68,11 +69,13 @@ defmodule EctoTrail do
 
       Insert arguments, return and options same as `c:Ecto.Repo.update/2` has.
       """
-      @spec update_and_log(changeset :: Ecto.Changeset.t,
-                           actor_id :: String.T,
-                           opts :: Keyword.t) ::
-            {:ok, Ecto.Schema.t} |
-            {:error, Ecto.Changeset.t}
+      @spec update_and_log(
+              changeset :: Ecto.Changeset.t(),
+              actor_id :: String.T,
+              opts :: Keyword.t()
+            ) ::
+              {:ok, Ecto.Schema.t()}
+              | {:error, Ecto.Changeset.t()}
       def update_and_log(changeset, actor_id, opts \\ []),
         do: EctoTrail.update_and_log(__MODULE__, changeset, actor_id, opts)
     end
@@ -83,13 +86,14 @@ defmodule EctoTrail do
 
   Insert arguments, return and options same as `c:Ecto.Repo.insert/2` has.
   """
-  @spec insert_and_log(repo :: Ecto.Repo.t,
-                       struct_or_changeset :: Ecto.Schema.t | Ecto.Changeset.t,
-                       actor_id :: String.T,
-                       opts :: Keyword.t) ::
-        {:ok, Ecto.Schema.t} | {:error, Ecto.Changeset.t}
+  @spec insert_and_log(
+          repo :: Ecto.Repo.t(),
+          struct_or_changeset :: Ecto.Schema.t() | Ecto.Changeset.t(),
+          actor_id :: String.T,
+          opts :: Keyword.t()
+        ) :: {:ok, Ecto.Schema.t()} | {:error, Ecto.Changeset.t()}
   def insert_and_log(repo, struct_or_changeset, actor_id, opts \\ []) do
-    Multi.new
+    Multi.new()
     |> Multi.insert(:operation, struct_or_changeset, opts)
     |> run_logging_transaction(repo, struct_or_changeset, actor_id)
   end
@@ -99,14 +103,16 @@ defmodule EctoTrail do
 
   Insert arguments, return and options same as `c:Ecto.Repo.update/2` has.
   """
-  @spec update_and_log(repo :: Ecto.Repo.t,
-                       changeset :: Ecto.Changeset.t,
-                       actor_id :: String.T,
-                       opts :: Keyword.t) ::
-        {:ok, Ecto.Schema.t} |
-        {:error, Ecto.Changeset.t}
+  @spec update_and_log(
+          repo :: Ecto.Repo.t(),
+          changeset :: Ecto.Changeset.t(),
+          actor_id :: String.T,
+          opts :: Keyword.t()
+        ) ::
+          {:ok, Ecto.Schema.t()}
+          | {:error, Ecto.Changeset.t()}
   def update_and_log(repo, changeset, actor_id, opts \\ []) do
-    Multi.new
+    Multi.new()
     |> Multi.update(:operation, changeset, opts)
     |> run_logging_transaction(repo, changeset, actor_id)
   end
@@ -120,6 +126,7 @@ defmodule EctoTrail do
 
   defp build_result({:ok, %{operation: operation}}),
     do: {:ok, operation}
+
   defp build_result({:error, :operation, reason, _changes_so_far}),
     do: {:error, reason}
 
@@ -148,25 +155,35 @@ defmodule EctoTrail do
     case result do
       {:ok, changelog} ->
         {:ok, changelog}
+
       {:error, reason} ->
-        Logger.error("Failed to store changes in audit log: #{inspect struct_or_changeset} " <>
-                     "by actor #{inspect actor_id}. Reason: #{inspect reason}")
+        Logger.error(
+          "Failed to store changes in audit log: #{inspect(struct_or_changeset)} " <>
+            "by actor #{inspect(actor_id)}. Reason: #{inspect(reason)}"
+        )
+
         {:ok, reason}
     end
   end
 
   defp get_changes(%Changeset{changes: changes}),
     do: map_custom_ecto_types(changes)
+
   defp get_changes(changes) when is_map(changes),
     do: changes |> Changeset.change(%{}) |> get_changes()
+
   defp get_changes(changes) when is_list(changes),
-    do: changes |> Enum.map_reduce([], fn(ch, acc) -> {nil, List.insert_at(acc, -1, get_changes(ch))} end) |> elem(1)
+    do:
+      changes
+      |> Enum.map_reduce([], fn ch, acc -> {nil, List.insert_at(acc, -1, get_changes(ch))} end)
+      |> elem(1)
 
   defp get_embed_changes(changeset, embeds) do
     Enum.reduce(embeds, changeset, fn embed, changeset ->
       case Map.get(changeset, embed) do
         nil ->
           changeset
+
         embed_changes ->
           Map.put(changeset, embed, get_changes(embed_changes))
       end
@@ -178,6 +195,7 @@ defmodule EctoTrail do
       case Map.get(changeset, assoc) do
         nil ->
           changeset
+
         assoc_changes ->
           Map.put(changeset, assoc, get_changes(assoc_changes))
       end
@@ -204,7 +222,7 @@ defmodule EctoTrail do
       :actor_id,
       :resource,
       :resource_id,
-      :changeset,
+      :changeset
     ])
   end
 end

mix.exs

@@ -4,19 +4,21 @@ defmodule EctoTrail.Mixfile do
   @version "0.2.4"
 
   def project do
-    [app: :ecto_trail,
-     description: description(),
-     package: package(),
-     version: @version,
-     elixir: "~> 1.4",
-     elixirc_paths: elixirc_paths(Mix.env),
-     compilers: [] ++ Mix.compilers,
-     build_embedded: Mix.env == :prod,
-     start_permanent: Mix.env == :prod,
-     deps: deps(),
-     test_coverage: [tool: ExCoveralls],
-     preferred_cli_env: [coveralls: :test],
-     docs: [source_ref: "v#\{@version\}", main: "readme", extras: ["README.md"]]]
+    [
+      app: :ecto_trail,
+      description: description(),
+      package: package(),
+      version: @version,
+      elixir: "~> 1.4",
+      elixirc_paths: elixirc_paths(Mix.env()),
+      compilers: [] ++ Mix.compilers(),
+      build_embedded: Mix.env() == :prod,
+      start_permanent: Mix.env() == :prod,
+      deps: deps(),
+      test_coverage: [tool: ExCoveralls],
+      preferred_cli_env: [coveralls: :test],
+      docs: [source_ref: "v#\{@version\}", main: "readme", extras: ["README.md"]]
+    ]
   end
 
   def description do
@@ -28,24 +30,28 @@ defmodule EctoTrail.Mixfile do
   end
 
   defp elixirc_paths(:test), do: ["lib", "test/support"]
-  defp elixirc_paths(_),     do: ["lib"]
+  defp elixirc_paths(_), do: ["lib"]
 
   defp deps do
-    [{:ecto, "~> 2.1"},
-     {:postgrex, "~> 0.13.2", optional: true},
-     {:dialyxir, "~> 0.5", only: [:dev, :test], runtime: false},
-     {:geo, "~> 1.4", only: [:dev, :test]},
-     {:ex_doc, ">= 0.15.0", only: [:dev, :test]},
-     {:excoveralls, ">= 0.5.0", only: [:dev, :test]},
-     {:dogma, ">= 0.1.12", only: [:dev, :test]},
-     {:credo, ">= 0.5.1", only: [:dev, :test]}]
+    [
+      {:ecto, "~> 2.1"},
+      {:postgrex, "~> 0.13.2", optional: true},
+      {:dialyxir, "~> 0.5", only: [:dev, :test], runtime: false},
+      {:geo, "~> 1.4", only: [:dev, :test]},
+      {:ex_doc, ">= 0.15.0", only: [:dev, :test]},
+      {:excoveralls, ">= 0.5.0", only: [:dev, :test]},
+      {:dogma, ">= 0.1.12", only: [:dev, :test]},
+      {:credo, ">= 0.5.1", only: [:dev, :test]}
+    ]
   end
 
   defp package do
-    [contributors: ["Nebo #15"],
-     maintainers: ["Nebo #15"],
-     licenses: ["LISENSE.md"],
-     links: %{github: "https://github.com/Nebo15/ecto_trail"},
-     files: ~w(lib LICENSE.md mix.exs README.md)]
+    [
+      contributors: ["Nebo #15"],
+      maintainers: ["Nebo #15"],
+      licenses: ["LISENSE.md"],
+      links: %{github: "https://github.com/Nebo15/ecto_trail"},
+      files: ~w(lib LICENSE.md mix.exs README.md)
+    ]
   end
 end

priv/repo/migrations/20170419082821_create_log_changes_table.exs

@@ -5,8 +5,7 @@ defmodule EctoTrail.TestRepo.Migrations.CreateAuditLogTable do
   @table_name String.to_atom(Application.fetch_env!(:ecto_trail, :table_name))
 
   def change(table_name \\ @table_name) do
-    create table(table_name, primary_key: false) do
-      add :id, :uuid, primary_key: true
+    create table(table_name) do
       add :actor_id, :string, null: false
       add :resource, :string, null: false
       add :resource_id, :string, null: false

pull_request_template.md

@@ -0,0 +1,38 @@
+# Description
+
+Please include a summary of the change and/or which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
+
+Fixes # (issue)
+
+## Type of change
+
+Please delete options that are not relevant.
+
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
+- [ ] This change requires a documentation update
+
+# How Has This Been Tested?
+
+Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
+
+- [ ] Test A
+- [ ] Test B
+
+**Test Configuration**:
+* Firmware version:
+* Hardware:
+* Toolchain:
+* SDK:
+
+# Checklist:
+
+- [ ] My code follows the style guidelines of this project
+- [ ] I have performed a self-review of my own code
+- [ ] I have commented my code, particularly in hard-to-understand areas
+- [ ] I have made corresponding changes to the documentation
+- [ ] My changes generate no new warnings
+- [ ] I have added tests that prove my fix is effective or that my feature works
+- [ ] New and existing unit tests pass locally with my changes
+- [ ] Any dependent changes have been merged and published in downstream modules