heywhy / ex_elasticlunr

Elasticlunr is a small, full-text search library for use in the Elixir environment. It indexes JSON documents and provides a friendly search interface to retrieve documents.
https://hexdocs.pm/elasticlunr
MIT License
189 stars 9 forks source link

replace uuid with uniq #17

Closed jdewar closed 2 years ago

jdewar commented 2 years ago

Overview

We noticed naming conflicts with :uuid and noticed it is suggested to replace it with :elixir_uuid by its maintainer:

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

However, this library still has the same errors as #15, which is not fixed in the current elixir_uuid (there are issues/PRs up to do so, and have been for some time). An alternate uuid library :uniq does not have these compilation warnings and is more up to date.

Related Issues

fixes #15

Testing

Tests run to completion with the new library and aliases.

heywhy commented 2 years ago

@jdewar Thank you for these changes. 👍