Add CI integration - Githubissues

luc-tielen commented 6 years ago

Hi,

I added some CI integrations to make developing easier. Features in this PR:

Fix failing tests / warnings in tests
mix credo --strict: linting
Add Travis CI integration for CI jobs
Add coveralls.io integration for test coverage
mix test.watch: run tests continuously on file change => nice for TDD
Makefile for doing common tasks when developing

Actions needed on your side:

https://travis-ci.org/ needs to be enabled for koudelka/visualixir (I have it enabled for my fork, should be just flipping a switch and then merging this PR)
https://coveralls.io/ also needs to be enabled for coverage information

For now, I left out dialyzer.. but I think this is an improvement already. Now with the better testing support, maybe a next PR could be made with extra tests etc?

luc-tielen commented 6 years ago

Also, after this is done, I think you can even let TravisCI check your PRs on this repo so they first need to pass linting / tests / ...

Just an idea.

luc-tielen commented 6 years ago

Any update on this? Would be nice to have this merged into master before I start adding tests etc. Otherwise I need to start branching from this branch 😅

koudelka commented 6 years ago

Sorry dude, I'm traveling at the moment.

I'm not entirely sure this project needs tests, since it's just a toy. I'm probably going to be changing some things around, it might be a waste of your time. :\

luc-tielen commented 6 years ago

No problem. I can always keep my fork around. Just saying it would be nice to have everything in one place. Reason I want the tests is for some changes I still want to make in the future.. In case you change your mind, feel free to merge in.

koudelka / visualixir

Add CI integration #19