[tahoe-lafs-trac-stream] [tahoe-lafs] #2052: Automate testing of merge requests to streamline review.
tahoe-lafs
trac at tahoe-lafs.org
Sat Aug 31 15:05:57 UTC 2013
#2052: Automate testing of merge requests to streamline review.
--------------------------+-----------------------------------------
Reporter: nejucomo | Owner: daira
Type: defect | Status: new
Priority: normal | Milestone: undecided
Component: unknown | Version: 1.10.0
Resolution: | Keywords: dev-infrastructure buildbot
Launchpad Bug: |
--------------------------+-----------------------------------------
Comment (by zooko):
I had trouble understanding comment:15 until I went ahead and read
http://about.travis-ci.org/blog/announcing-pull-request-support/ . Now it
is all clear! Here is the desired functionality (desired at least by
amontero and by me):
1. A new/casual contributor clicks the "fork this" button on github.
2. The contributor write a patch and pushes it to their repo on github.
3. The contributor pushes the "pull request" button on github.
4. Very soon, and with no manual intervention from any Tahoe-LAFS
developer, some automated bot somewhere runs the unit tests on the version
in the pull request, and posts a comment on the pull request indicating
whether the tests pass or not, and including a link to the test results.
This is exactly what I want, and apparently other people also want this
and have already implemented it, using travis-ci. It looks like we might
be able to get the travis-ci people to do this for us and avoid all the
problems of implementing it ourselves in our buildbot scripts, and the
problems of security of our buildslaves.
I intend to investigate signing up for this. It looks like, from the end
of http://about.travis-ci.org/blog/announcing-pull-request-support/, that
you have to pay ("donate") to the travis-ci project in order to get in on
this.
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2052#comment:16>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list