search

SodaqMoja / SodaqOne-UniversalTracker-v2

6 stars 15 forks source link

Add travis-ci configuration #3

Closed jwillemsen closed 7 years ago

jwillemsen commented 7 years ago

Adding a travis-ci configuration, please open an account on travis-ci.org so that there is a CI. After that a badge can be added to the readme.md

jwillemsen commented 7 years ago

WIth this travis file as example maybe to all sodaqmoja repo's a travis-ci configuration can be added? Hopefully at some point platformio fixes the warning and we could enable -Werror

jwillemsen commented 7 years ago

Any plans to merge this?

keestux commented 7 years ago

Can you do a squash?

jwillemsen commented 7 years ago

Github has a squash and merge option for pull requests, maybe you have to allow it in the project settings? That is the easiest to do

keestux commented 7 years ago

When I asked for a squash I was expecting that you do it in your own local repo. And then do a forced push to update the PR branch. (( That's how we do things in RIOT-OS, for example. ))

jwillemsen commented 7 years ago

Can do that tomorrow, personally I don't like forced updates but just use the squash commit feature from github

jwillemsen commented 7 years ago

@keestux Done, squashed to one change