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

fix various compile warnings #253

Open
tttp opened this issue Jun 13, 2024 · 1 comment
Open

fix various compile warnings #253

tttp opened this issue Jun 13, 2024 · 1 comment
Assignees

Comments

@tttp
Copy link
Member

tttp commented Jun 13, 2024

Low priority, just flagging now that I rebuild regularly ;)

warning: use Mix.Config is deprecated. Use the Config module instead

@destag
Copy link
Contributor

destag commented Jun 13, 2024

We get even more warnings if this is the first time we compile applications. I think we can make a collective PR with fixing compile warnings

❯ mix compile --all-warnings 2>&1 | grep 'warning:' | wc -l
114

@tttp tttp changed the title replace mix.config by config module fix various compile warnings Jun 13, 2024
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

When branches are created from issues, their pull requests are automatically linked.

2 participants