Commits (13)
-
Oswald Buddenhagen authored
whitelisting is supported as well Change-Id: I852ca6b70bf2424732a4b252ba9ce959e2794485 Reviewed-by:
Sergio Ahumada <sahumada@texla.cl>
eadd183e -
Samuel Gaist authored
Currently there's a mix between bash and sh in the scripts. This patch aims to unify the use of sh since there's no code involving bash specific features Change-Id: I30fdae59511b72910facf5e5be3bb9827f4f537c Reviewed-by:
Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
31ec5e51 -
Oswald Buddenhagen authored
quote it right before building a command line. Change-Id: I5bd95dbe2fc392b974b730189277a62ae6ae59d5 Reviewed-by:
Orgad Shaneh <orgads@gmail.com>
00bd3734 -
Oswald Buddenhagen authored
conceivably, this mechanism could be used in more cases, for example on -2 reviews. however, this would need some additional tagging by the worker script - every WIP is a -2 ... Change-Id: Ib10777089926164b70dc3b7dd46c2fb526ecf165 Reviewed-by:
Orgad Shaneh <orgads@gmail.com>
2502d4cf -
Oswald Buddenhagen authored
make the regexes for qmake more strict. on top of that, zap the matched strings, so they don't trigger the terminology warning as well. Change-Id: I0404c49ad55d2d1565828d78fca85108e792e20e Reviewed-by:
Jake Petroules <jake.petroules@petroules.com>
822a52eb -
Oswald Buddenhagen authored
there is no reason to restrict them to ChangeLog surroundings. also complain about any regular text following footers, as that is just bad style. Change-Id: Id3a0d07eb9d45d27629c82451713d8cea9a68a09 Reviewed-by:
Orgad Shaneh <orgads@gmail.com>
d7bfe46b -
Oswald Buddenhagen authored
watchers are really pointless when sanitizing local commits, especially as nobody actually has the relevant configuration. as an additional boon, the bot can now automatically invite relevant people. Change-Id: I735056845d7f807904070e61efb54407dcec9fb9 Reviewed-by:
Orgad Shaneh <orgads@gmail.com> Reviewed-by:
Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
f097355b -
Sergio Ahumada authored
Change-Id: I4614ad0e1e2d5d0743a4ff5ebe24eea406f92f9b Reviewed-by:
Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
4667f1fb -
Oswald Buddenhagen authored
it's pointless (the doc says it's basically internal) and breaks msysgit's perl. Change-Id: Ia2d728dce57d6a447a720622a716c5deab0e7595 Reviewed-by:
Orgad Shaneh <orgads@gmail.com>
12c11eba -
Simon Hausmann authored
Go source code managed via go fmt mandates tabs Change-Id: I9cbefcd7b7bb959b22aaa10937189d086d368e9e Reviewed-by:
Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
5bc8acfd -
Jani Heikkinen authored
Qt copyrights are now in The Qt Company, so we could update the source code headers accordingly. In the same go we should also fix the links to point to qt.io. Change-Id: If3c8d221bcf8e5010048ed5f1c50cd35c04e7199 Reviewed-by:
Jani Heikkinen <jani.heikkinen@theqtcompany.com> Reviewed-by:
Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
48e5fd92 -
Oswald Buddenhagen authored
they are fixed terminology, and especially in the case of 'Changes' it's sometimes confusing without some marker. Change-Id: I75ce8d65940ea2f373cb86488fc4ac2aac2f9fde Reviewed-by:
Sergio Ahumada <sahumada@texla.cl>
62a2168e -
Oswald Buddenhagen authored
it obviously makes no sense when the sanitizer is executed locally. additionally, this allows us a different treatment of WIP reports: instead of giving a -2 sanity review, give a -2 code review. that way the change owner is not spammed by useless notifications (because the sanity review itself is positive, so the message is suppressed by the gerrit configuration). Change-Id: I10bc8f8f61673e75983dcb91310836d59c434483 Reviewed-by:
Orgad Shaneh <orgads@gmail.com> Reviewed-by:
Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
cbddd3a5
Showing