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

Add CI with code coverage using Travis CI and Codecov #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mpeterv
Copy link
Contributor

@mpeterv mpeterv commented Apr 29, 2018

This adds configuration for Travis CI to run t/test.lua on each commit under Lua 5.1, 5.2, 5.3, and LuaJIT 2.0, 2.1.

It looks like this: https://travis-ci.org/mpeterv/serpent

Additionally, test coverage is collected using Luacov and uploaded to Codecov. Looks like this: https://codecov.io/gh/mpeterv/serpent/branch/ci

Coverage is pretty good, only two if branches are not touched by existing tests.

Also, add nice badges to readme to make it easy to see if the build if failing currently and what is the current coverage percentage.

Before applying this, need to register on travis-ci and add the repo. Also need to join Codecov, but no need to add the repo manually there.

@pkulchenko
Copy link
Owner

Looks good; thank you for the patch! I'll indeed have to register with travis and codecov as you said; plan on doing this soon.

@pkulchenko pkulchenko self-assigned this Apr 30, 2018
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