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

Running mix xref trace triggers a consolidation warning #11899

Closed
DavidVII opened this issue Jun 3, 2022 · 0 comments
Closed

Running mix xref trace triggers a consolidation warning #11899

DavidVII opened this issue Jun 3, 2022 · 0 comments

Comments

@DavidVII
Copy link

DavidVII commented Jun 3, 2022

Elixir and Erlang/OTP versions

Elixir 1.13.4 (compiled with Erlang/OTP 24)

Operating System

MacOS 12.4

Current behavior

I'm noticing this issue after upgrading to Elixir 1.12.3 -> 1.13.4 and Phoenix 1.6.6 -> 1.6.10 when I run the following:

$ mix xref trace lib/ros/accounts/user.ex
> warning: the Inspect protocol has already been consolidated, an implementation for Ros.Accounts.User has no effect. If you want to implement protocols after compilation or during tests, check the "Consolidation" section in the Protocol module documentation
  lib/ros/accounts/user.ex:11: Ros.Accounts.User (module)

The referenced code looks like this:

10  @derive {Inspect, except: [:password]}
11  schema "users" do

I've read the Consolidation section and it's unclear to me if this is an actual issue or if I can safely ignore it.

Expected behavior

It was mentioned here that this is safe, but that we should either document this warning or update mix trace to avoid this warning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants