Skip to content

Compatibility with Ecto ~> 3.2, ULID.t, formatter#10

Closed
woylie wants to merge 8 commits intoTheRealReal:masterfrom
woylie:ecto-compatibility
Closed

Compatibility with Ecto ~> 3.2, ULID.t, formatter#10
woylie wants to merge 8 commits intoTheRealReal:masterfrom
woylie:ecto-compatibility

Conversation

@woylie
Copy link

@woylie woylie commented Nov 16, 2020

I needed this to work, so I created my own fork and cherry-picked what I needed and made some changes to make it work on the CI.

  • fixes compatibility issue with Ecto ~> 3.2 (cherry-picked from Fix warning when using ecto 3.5. #7)
  • adds ULID type (from Adding type t #6)
  • bumps Ecto requirement to ~> 3.2
  • requires Elixir 1.7 (because of newer Ecto versions)
  • updates CI configuration
  • formats code with mix format

@warmwaffles
Copy link

What is the status on this @TheRealReal?

@ykurtbas
Copy link

@woylie thank you, I am using your version, would rather not checkout a git repo though. What is the status?

@woylie
Copy link
Author

woylie commented Mar 10, 2021

@ykurtbas Since it doesn't seem like anything is going to happen here anymore, I just published my fork as https://hex.pm/packages/ecto_ulid_next.

@ykurtbas
Copy link

@ykurtbas Since it doesn't seem like anything is going to happen here anymore, I just published my fork as https://hex.pm/packages/ecto_ulid_next.

Thank you! I will happily point to that and tick off one more git reference :)

@woylie woylie closed this Mar 11, 2021
@woylie woylie deleted the ecto-compatibility branch March 11, 2021 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants