@@ -21,9 +21,64 @@ This edition covers what happened during the month of September 2015.
2121### General
2222-->
2323
24- <!-- -
2524### Reviews
26- -->
25+
26+ * [ Add Travis CI support] ( http://git.661346.n2.nabble.com/RFC-PATCH-v1-Add-Travis-CI-support-tt7640334.html )
27+
28+ Lars Schneider wrote that to avoid breaking tests in the future he
29+ configured Travis-CI to run all Git tests including git-p4 and git-lfs
30+ tests on both Ubuntu and MacOS using gcc and clang.
31+
32+ If his config was enabled on https://github.com/git/git then the
33+ status of all official branches and all pull requests could be known
34+ by any contributor for free.
35+
36+ Junio, the Git maintainer, replied that the last time he looked at it,
37+ Travis wanted write access to the repository, and that for security
38+ reason he could not allow that.
39+
40+ Dennis Kaarsemaker replied with the following:
41+
42+ > It does not need write access to the git data, only to auxiliary
43+ > GitHub data: commit status and deployment status (where it can put
44+ > "this commit failed tests"), repository hooks (to set up build
45+ > triggers), team membership (ro) and email addresses (ro).
46+
47+ And then people started discussing if it would be better for Travis to
48+ be configured only on a fork of git/git and about the annoyance that
49+ emails sent by Travis might be.
50+
51+ Roberto Tyley, the author of submitGit, then talked about possible
52+ enhancement to the submitGit workflow if Travis or another CI system
53+ are set up, like enabling sending a patch email to the Git mailing
54+ list only after the test results are available.
55+
56+ And then Matthieu Moy explained the benefits for everyone like this:
57+
58+ > The very nice thing with Travis-CI is that it does not only test the
59+ > repository's branches, but also all pull-requests. So, if it is
60+ > activated on git/git, it will become possible to have a flow like
61+ >
62+ > 1 ) User pushes to his own repo, sends a pull-request,
63+ >
64+ > 2 ) Travis-CI notices the pull-request and builds it (no action needed
65+ > from anyone),
66+ >
67+ > 3 ) Once the build is finished, the user can use e.g. SubmitGit to
68+ > actually submit the code.
69+ >
70+ > This has real benefits for the submitter (know if your code is broken
71+ > early), for the reviewers (things like "you have a def-after-use" would
72+ > be noticed by a computer before human beings start spending time on the
73+ > review), and for you (some issues noticed before a topic enters pu).
74+ >
75+ > There's no extra work for the user at all compared to the standard
76+ > pull-request flow (nothing to do, just submit a PR), and a one-time
77+ > setup for the project.
78+
79+ This appear to have convinced Junio of the value of a CI tool linked
80+ to git/git, so an interesting way to test patchs will perhaps be
81+ available soon to Git developers.
2782
2883<!-- -
2984### Support
0 commit comments