lalabuy948 / PhoenixAnalytics

📊 Plug and play analytics for Phoenix applications.
https://theindiestack.com/analytics
Apache License 2.0
270 stars 10 forks source link

Consider to use `elixir_uuid` instead of `uuid` #15

Closed vermaxik closed 2 months ago

vermaxik commented 2 months ago

Thanks for your library, looks really great, tried to use in the dockerized app with mix release, but didn't have a luck with package name coalitions, because already have elixir_uuid from another package.

using config/runtime.exs to configure the release at runtime
[13:30:03.523] #19 1.850 ** (Mix) Duplicated modules:
[13:30:03.532] #19 1.850    'Elixir.UUID' specified in elixir_uuid and uuid
[13:30:03.632] #19 ERROR: process "/bin/sh -c mix release" did not complete successfully: exit code: 1

Would it possible to upgrade the uuid to elixir_uuid, as recommended https://github.com/zyro/elixir-uuid?

Note: Renamed from uuid to elixir_uuid as a solution to package name conflicts. Use elixir_uuid going forward.

Let me know if I should create PR. Thank you.

lalabuy948 commented 2 months ago

Hi @vermaxik ,

Thank you for reporting this issue. Could you be so kind to take a look on this PR?

lalabuy948 commented 2 months ago

Will close this one. Resoled in 0.1.3