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 depcrecation warnings & CI #34

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

paulgoetze
Copy link
Member

@paulgoetze paulgoetze commented Mar 5, 2023

  • Updates the CI to use the latest working Elixir (1.13.4) (1.14. is not yet working, see Compilation error with Elixir 1.14.0 deadtrickster/prometheus.ex#48)

  • Fixes the deprecation warning during compilation:

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

    and during test runs:

    warning: the @initial_state() notation (with parenthesis) is deprecated, please use @initial_state (without parenthesis) instead
    

@paulgoetze paulgoetze force-pushed the fix/mix-config-deprecation-warning branch from 1e4b3e4 to 4950c8d Compare March 5, 2023 16:01
@paulgoetze paulgoetze force-pushed the fix/mix-config-deprecation-warning branch from 4950c8d to cdc5517 Compare March 5, 2023 16:20
warning: the @initial_state() notation (with parenthesis) is deprecated,
         please use @initial_state (without parenthesis) instead
@paulgoetze paulgoetze changed the title Fix depcrecation warning: import Config instead of use Mix.Config Fix depcrecation warnings & CI Mar 5, 2023
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.

1 participant