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

Added credo dependency for Elixir code analysis. #44

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

Conversation

RafaelFriesen
Copy link

Credo is a static code analysis tool for the Elixir language with a focus on teaching and code consistency. More documentation can be found at https://github.com/rrrene/credo

Credo is a static code analysis tool for the Elixir language with a focus on teaching and code consistency. More documentation can be found at https://github.com/rrrene/credo
@RafaelFriesen RafaelFriesen changed the title Added credo dependency. Added credo dependency for Elixir code analysis. Jan 26, 2016
I will play around with this file to find out how to deploy credo best.
Moved credo command from 'script' to 'before_install' to avoid breaking the test with credo's return 31 or 1. Credo needs deps.get, so I do it twice.
Moved credo call, still looking for the right place.
@GalaxyGorilla
Copy link
Contributor

113 mods/funs, found 2 consistency issues, 33 warnings, 51 refactoring opportunities, 10 code readability issues, 2 software design suggestions.

@RafaelFriesen: Feel free to provide improvements such that the CI tests will not fail anymore ;)

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.

2 participants