improvement: allow for combining AshPostgres.Repo with other repos

fixes #172

Author: zachdaniel

lib/data_layer.ex

@@ -391,7 +391,6 @@ defmodule AshPostgres.DataLayer do
     sections: @sections,
     transformers: [
       AshPostgres.Transformers.ValidateReferences,
-      AshPostgres.Transformers.VerifyRepo,
       AshPostgres.Transformers.EnsureTableOrPolymorphic,
       AshPostgres.Transformers.PreventMultidimensionalArrayAggregates
     ]
@@ -502,7 +501,7 @@ defmodule AshPostgres.DataLayer do
   def can?(_, :timeout), do: true
   def can?(_, {:filter_expr, _}), do: true
   def can?(_, :nested_expressions), do: true
-  def can?(_, {:query_aggregate, :count}), do: true
+  def can?(_, {:query_aggregate, _}), do: true
   def can?(_, :sort), do: true
   def can?(_, :distinct_sort), do: true
   def can?(_, :distinct), do: true

lib/repo.ex

@@ -45,6 +45,11 @@ defmodule AshPostgres.Repo do
   Use this to inform the data layer about the oldest potential postgres version it will be run on.
 
   Must be an integer greater than or equal to 13.
+
+  ## Combining with other tools
+
+  For things like `Fly.Repo`, where you might need to have more fine grained control over the repo module,
+  you can use the `define_ecto_repo?: false` option to `use AshPostgres.Repo`.
   """
   @callback min_pg_version() :: integer()
 
@@ -63,11 +68,13 @@ defmodule AshPostgres.Repo do
 
   defmacro __using__(opts) do
     quote bind_quoted: [opts: opts] do
-      otp_app = opts[:otp_app] || raise("Must configure OTP app")
+      if Keyword.get(opts, :define_ecto_repo?, true) do
+        otp_app = opts[:otp_app] || raise("Must configure OTP app")
 
-      use Ecto.Repo,
-        adapter: Ecto.Adapters.Postgres,
-        otp_app: otp_app
+        use Ecto.Repo,
+          adapter: Ecto.Adapters.Postgres,
+          otp_app: otp_app
+      end
 
       @behaviour AshPostgres.Repo