Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds :crypto as a dependency in extra_applications #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vrcca
Copy link

@vrcca vrcca commented Oct 7, 2020

This is in order to stop Elixir 1.11 warning about undeclared dependencies.

==> elixir_uuid
Compiling 1 file (.ex)
warning: :crypto.hash/2 defined in application :crypto is used by the current application but the current application does not directly depend on :crypto. To fix this, you must do one of:

  1. If :crypto is part of Erlang/Elixir, you must include it under :extra_applications inside "def application" in your mix.exs

  2. If :crypto is a dependency, make sure it is listed under "def deps" in your mix.exs

  3. In case you don't want to add a requirement to :crypto, you may optionally skip this warning by adding [xref: [exclude: :crypto] to your "def project" in mix.exs

Found at 2 locations:
  lib/uuid.ex:589: UUID.namebased_uuid/2
  lib/uuid.ex:593: UUID.namebased_uuid/2

This is in order to stop Elixir 1.11 warning about undeclared dependencies.
@WannesFransen1994
Copy link

Thanks for the hard work! Hope this will be merged soon.

@jrissler
Copy link

jrissler commented Feb 25, 2021

@zyro can we get this merged?

@shadiramadan
Copy link

@zyro It's been more than a year that this PR has been up. Do you no longer maintain this repo / do you need more maintainers?

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.

4 participants