
<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi<div class=""><br class=""><blockquote type="cite" class="">On 02 Jun 2015, at 02:24, Matthias Kuhn <<a href="mailto:matthias@opengis.ch" class="">matthias@opengis.ch</a>> wrote:<br class=""><br class="">Hi,<br class=""><br class="">For more than a week already we have failing tests on master<br class=""><a href="https://travis-ci.org/qgis/QGIS/builds" class="">https://travis-ci.org/qgis/QGIS/builds</a><br class=""><br class="">This basically renders the whole test suite a lot less usable.<br class=""><br class="">Some examples/thoughts:<br class=""><br class="">* Every pull request fails tests [1] I often look at the test results<br class="">to check if it's even worth looking at a pull request. Handling pull<br class="">requests is an issue on its own as there is no direct benefit for the<br class="">reviewer most of the time. Don't make the reviewer's time even harder!<br class=""><br class="">* We don't see when a commit trashes our compiler. Today we had a<br class="">commit which broke the compiler. The author was not informed by mail.<br class=""><br class="">* We don't see which commit broke a certain test. E.g. At the moment<br class="">the PyQgsAppStartup test is failing. And to find out which commit broke<br class="">it one has to dig deep because failing of this test is masked by the<br class="">already failing tests.<br class=""><br class="">* It discourages people to write tests if they can be broken easily.<br class=""><br class="">The tests started to fail with the merge of some big work, just previous<br class="">to feature freeze. I can understand that people want to get commits into<br class="">master before feature freeze. But quality should not suffer from this.<br class="">And if there's something broken, please fix it fast.<br class=""><br class="">Finally Nyall was starting to fix things - crucial GIS functionality<br class="">like splitting features - although I don't think he received anything in<br class="">return, thank you very much, Nyall. The one thing left to do is to add<br class="">doxygen API documentation - and to fix the newly broken PyQgsAppStartup<br class="">test. I really hope we will finally see the green light again.<br class=""><br class="">I am a hippy. I don't like rules and policies. I am not in favor of a<br class="">very strong policy concerning the tests. Common sense should be valued<br class="">more than rules. Policies like "every new feature has to come with a<br class="">test" are no good solution as they encourage people to write silly<br class="">tests. And I am much too lazy to write a QEP for a policy to revert<br class="">commits which break the tests.<br class=""><br class="">So... How do we fix it?<br class="">If tomorrow everybody on this list writes some API documentation we<br class="">should have fixed that test by noon.<br class="">And then I'll promise I'll fix the failing PyQgsAppStartup test to get<br class="">the green light going again.<br class=""><br class="">Thank you very much<br class="">Matthias<br class=""><br class="">[1] https://github.com/qgis/QGIS/pulls ;<br class="">https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aclosed<br class="">[2]<br class="">https://github.com/qgis/QGIS/commit/ded11b32562c100aeaae95bf20e21bcfc38d0777<br class=""></blockquote><div class=""><br class=""></div><div class="">I can only wholeheartedly second Matthias’ message above - we should never push to master if the tests don’t pass, and if for some reason they break we should not push anything more to master before the break is fixed.</div><div class=""><br class=""></div><div class="">Regards</div><div class=""><br class=""></div><div class="">Tim</div><div class=""><br class=""></div><br class=""><blockquote type="cite" class=""><br class="">_______________________________________________<br class="">Qgis-developer mailing list<br class=""><a href="mailto:Qgis-developer@lists.osgeo.org" class="">Qgis-developer@lists.osgeo.org</a><br class="">http://lists.osgeo.org/mailman/listinfo/qgis-developer<br class=""></blockquote><br class=""><div class=""><span>—</span><br class=""><span><br class=""></span><span style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br class="Apple-interchange-newline"><span><img height="66" width="160" apple-inline="yes" id="0D2F8CCA-E0AA-4DD0-8415-A12099599357" apple-width="yes" apple-height="yes" src="cid:62C890D4-3964-4609-BDE6-7536D5FBDD70" class=""></span><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-align: center;" class=""><br class="Apple-interchange-newline"><br class=""></div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-align: center;" class="">Tim Sutton</div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-align: center;" class=""><br class=""></div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div style="text-align: center;" class="">Visit <a href="http://kartoza.com" class="">http://kartoza.com</a> to find out about open source:</div><div style="text-align: center;" class=""><br class=""></div><div class=""><div style="text-align: center;" class="">* Desktop GIS programming services</div><div style="text-align: center;" class="">* Geospatial web development</div><div style="text-align: center;" class="">* GIS Training</div><div style="text-align: center;" class="">* Consulting Services</div><div style="text-align: center;" class=""><br class=""></div><div class=""><div style="text-align: center;" class="">Skype: timlinux Irc: timlinux on #qgis at <a href="http://freenode.net" class="">freenode.net</a></div><div style="text-align: center;" class="">Tim is a member of the QGIS Project Steering Committee</div><div style="text-align: center;" class=""><br class=""></div><div style="text-align: center;" class="">Kartoza is a merger between Linfiniti and Afrispatial</div></div></div></div>

</span></div><br class=""></div></body></html>