monome / teletype

monome eurorack module
GNU General Public License v2.0
195 stars 82 forks source link

Setup travis build. #28

Closed pq closed 8 years ago

pq commented 8 years ago

A few bits:

pq commented 8 years ago

@tehn : looks like an admin needs to flip the bit on travis for teletype:

screen shot 2016-04-06 at 2 33 38 pm

https://travis-ci.org/profile/monome

Could you?

tehn commented 8 years ago

flipped

On Wed, Apr 6, 2016 at 5:35 PM, Phil Quitslund notifications@github.com wrote:

@tehn https://github.com/tehn : looks like an admin needs to flip the bit on travis for teletype:

[image: screen shot 2016-04-06 at 2 33 38 pm] https://cloud.githubusercontent.com/assets/67586/14333624/9900ad62-fc04-11e5-8d42-d97511c47598.png

https://travis-ci.org/profile/monome

Could you?

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/monome/teletype/issues/28#issuecomment-206581292

tehn commented 8 years ago

i must say, i am completely baffled by the contents of .travis.yml

On Wed, Apr 6, 2016 at 5:35 PM, Phil Quitslund notifications@github.com wrote:

@tehn https://github.com/tehn : looks like an admin needs to flip the bit on travis for teletype:

[image: screen shot 2016-04-06 at 2 33 38 pm] https://cloud.githubusercontent.com/assets/67586/14333624/9900ad62-fc04-11e5-8d42-d97511c47598.png

https://travis-ci.org/profile/monome

Could you?

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/monome/teletype/issues/28#issuecomment-206581292

pq commented 8 years ago

Ha! I blame @samdoshi.

tehn commented 8 years ago

ah, it makes more sense now that i actually read about travis.

so right now we're just doing a compilation check, correct?

On Wed, Apr 6, 2016 at 6:23 PM, Phil Quitslund notifications@github.com wrote:

Ha! I blame @samdoshi https://github.com/samdoshi.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/monome/teletype/issues/28#issuecomment-206594870

pq commented 8 years ago

so right now we're just doing a compilation check, correct?

Right. Once we have tests we should add them. Down the road we could add linting or some other kind of presubmit checks. Note that pull requests trigger a build which means you can see if a PR will break the build before accepting it.

For example:

screen shot 2016-04-06 at 4 02 27 pm

Which is pretty neat!

pq commented 8 years ago

Added badge with https://github.com/monome/teletype/commit/cbb9fe8eb7613dfa91ccdc9d16a9953af4165b02 which means this first bit is done.

@samdoshi or @tehn : feel free to re-open or create another issue if you want to see refinements to the travis build.

tehn commented 8 years ago

looks good, thanks!

samdoshi commented 8 years ago

i must say, i am completely baffled by the contents of .travis.yml

You and me both.

Should we get https://github.com/samdoshi/avr32-toolchain-linux moved over to the 'monome' org too?

pq commented 8 years ago

Should we get https://github.com/samdoshi/avr32-toolchain-linux moved over to the 'monome' org too?

:+1:

Make sense to me!