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

Integrate Credo into travis #18

Open
NobbZ opened this issue Mar 4, 2016 · 0 comments
Open

Integrate Credo into travis #18

NobbZ opened this issue Mar 4, 2016 · 0 comments

Comments

@NobbZ
Copy link
Collaborator

NobbZ commented Mar 4, 2016

Since we have decided in #12 that we want an error on errornous typestuff in dialyzer, I thought that we should think about also using credo in travis.

This would again mean, that emitted credo warnings will result in travis fails. Therefore I do think, that we should discuss this here, and regardless of the outcome of this discussion, we should not integrate it before we haven't finished repairing all the typestuff coming from dialyzer integration.

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

No branches or pull requests

1 participant