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

Revisit running dialyzer as part of CI #1300

Open
vanstee opened this issue Jan 11, 2017 · 0 comments
Open

Revisit running dialyzer as part of CI #1300

vanstee opened this issue Jan 11, 2017 · 0 comments

Comments

@vanstee
Copy link
Member

vanstee commented Jan 11, 2017

Elixir 1.3 included some improvements both for working with types and in its own spec definitions. After running dialyxir (1.4) the warnings emitted seem to be real warnings and not just wonky Elixir issues coming up to the surface, so let's give it a shot and see if we can add it as a final step to our CI builds.

  • Add dialyxir to Cog
  • Fix or ignore all warnings emitted
  • Run mix dialyzer --halt-exit-status as part of CI builds
@vanstee vanstee added the cog label Jan 11, 2017
@vanstee vanstee added this to the Cog 1.1 milestone Jan 11, 2017
@christophermaier christophermaier removed this from the Cog 1.1 milestone Aug 23, 2017
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

No branches or pull requests

2 participants