Conversation
|
@jayjun could you update the tests so CI passes? |
|
@markusfeyh Hi there, appreciate you looking into this. CI was failing because Travis couldn’t fetch certain Erlang versions. Fixed and expanded coverage up to Elixir 1.11. For your convenience, #7, #8 and half of #10 are duplicates. |
|
@TheRealReal @markusfeyh What's the status on this being merged? Another library I'm using is requiring Ecto 3.4, so I also needed to fork @jayjun's branchh to keep using |
|
could we please get an official version that supports Ecto 3.X branch? I am depending on people's forks. |
|
@ykurtbas merged and created a tag to reference: |
|
@markusfeyh Thanks for merging. Can you publish 0.3.0 to Hex now? See https://hex.pm/packages/ecto_ulid. |
Ecto 3.2.0 requires two new
Ecto.Typecallbacks,embed_as/1andequal?/2.use Ecto.Typewas also added to provide default implementations of those callbacks.This pull request adds those implementations to
Ecto.ULIDuntil Ecto 3.2.0 is required.