search

ryandotsmith / l2met

Convert a formatted log stream into metrics
http://r.32k.io/l2met-introduction
GNU General Public License v3.0
284 stars 26 forks source link

Travis integration #117

Closed josephruscio closed 11 years ago

josephruscio commented 11 years ago

Upstream l2met (this repo) should run tests on Travis.org.

ryandotsmith commented 11 years ago

I am curious as to your motivation. What problem would this solve?

FIWI: I use drone.io for the build/CI service on l2met because it is simpler and faster than travis. Last time I used travis for l2met, it took almost 5 minutes to execute a build. On drone.io it takes ~5s.

josephruscio commented 11 years ago

I filed this because I thought master in our private fork was failing tests. It turns out that Travis apparently will run tests on branches without a .travis.yml file, and assume they are Ruby :-1:. I'm going to close this because the original motivation isn't valid.

That aside, it would provide a public/automated indicator of the validity of master. If (drone.io or any other CI provider) has some Github integration that would be fine too. Food for thought ;-).

ryandotsmith commented 11 years ago

@josephruscio Did you see the badge under the hacking/testing section of the README?

https://github.com/ryandotsmith/l2met#running-tests

Maybe it would be better to place it at the top?

josephruscio commented 11 years ago

I did not see it and :+1: to placing it at the top. That seems to be the convention for important badges like CI or code quality.