Re: [RFC/PATCH v1] Add Travis CI support
- From
Luke Diamand <luke@diamand.org>
- Date
- Sep 25, 2015, 08:05 UTC
- Message-ID
- <CAE5ih7_f8qy9WvmgRUR6-qFwB4WFhZ6Qr5iOpE0YxqJH8AsZyw@mail.gmail.com>
- In-Reply-To
- <699c08632232180166145f70c7f16645@dscho.org>
On 25 September 2015 at 08:27, Johannes Schindelin <johannes.schindelin@gmx.de> wrote:
Show 22 quoted lines
> Hi, > > On 2015-09-25 05:14, Dennis Kaarsemaker wrote: >> On do, 2015-09-24 at 17:41 -0700, Junio C Hamano wrote: >>> larsxschneider@gmail.com writes: >>> >>> > My idea is that the owner of "https://github.com/git/git" enables this account >>> > for Travis (it's free!). Then we would automatically get the test state for all >>> > official branches. >>> >>> The last time I heard about this "it's free" thing, I thought I >>> heard that it wants write access to the repository. >> >> It does not need write access to the git data, only to auxiliary GitHub >> data: commit status and deployment status (where it can put "this >> commit failed tests"), repository hooks (to set up build triggers), >> team membership (ro) and email addresses (ro). > > If that still elicits concerns, a fork could be set up that is automatically kept up-to-date via a web hook, and enable Travis CI there. > > Junio, if that is something with which you feel more comfortable, I would be willing to set it up. Even if the visibility (read: impact) would be higher if the badges were attached to https://github.com/git/git proper... >
It would be less intrusive for the CI system to have a fork. Otherwise other people using git with the same CI system will get annoying merge conflicts, and we'll also end up with a repo littered with the control files from past CI systems if the CI system is ever changed.
>From past experience, if it's configured to email people when things
break, sooner or later it will email the wrong people, probably once every few seconds over a weekend.
Automated testing is a Good Thing, but it's still software, so needs maintenance or it will break.
Luke