summaryrefslogtreecommitdiffstats
path: root/git-hooks
Commit message (Collapse)AuthorAgeFilesLines
* put change refs into a more defined placeOswald Buddenhagen2012-06-081-7/+6
| | | | | | | don't mess with the remote ref string Change-Id: I30890801a40fc66049e96534ec69c7fe60032085 Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* make fetching mainlines more efficientOswald Buddenhagen2012-06-081-9/+16
| | | | | | | | - don't re-fetch them unless gerrit tells us there was an update - fetch them together with the change ref Change-Id: Icf00564ad676cd1b30cd5ab6596df83520d3fa53 Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* add cleanup script for sanity botOswald Buddenhagen2012-06-071-0/+24
| | | | | | | | | | | | if the sanity bot is configured with gitdofetch=1, the repos need a periodic cleanup - otherwise they become amazingly slow (a simple fetch can take minutes before anything happens). i was too lazy make this an option of the sanity-bot script itself. maybe later. Change-Id: I85306264cd67d321c10cb069e5bfb5b60b2ca6db Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* add possibility of persistent configurationOswald Buddenhagen2012-05-091-0/+16
| | | | | | | | | | | | | options are stored in as git options in the group sanity.<dirname>, with <dirname> being the basename of the repository. it is recommended that these keys are added to the global git config, outside the clones of the inspected repos. the only currently recognized option is "flags", which corresponds to the GIT_PUSH environment variable. Change-Id: I011d4f335ca94b5aa08fc9bcb6283e6c2f7b3b79 Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* allow suppressing tabs checkOswald Buddenhagen2012-05-091-1/+1
| | | | | | Change-Id: I586495e5b9d2561be6c8c16e6824a21af674f044 Reviewed-by: Sergio Ahumada <sergio.ahumada@nokia.com> Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* moan about adding backup filesOswald Buddenhagen2012-02-201-1/+4
| | | | | | Change-Id: I000026d8b78deef11245cdc939d9c06eebe676c0 Reviewed-by: Sergio Ahumada <sergio.ahumada@nokia.com> Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* exclude debian/rules from the tabs checkOswald Buddenhagen2012-02-201-1/+1
| | | | | | | | | | | these files are really Makefiles, so complaining about tabs in them is counterproductive. the regexp will catch debian.rules as well, which is the convention found in obs. Change-Id: I4487a964468f2d3fbf3cad9c91be06ebfdcfaefb Reviewed-by: Sergio Ahumada <sergio.ahumada@nokia.com> Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* make symlinking the script actually work without setting up PATHOswald Buddenhagen2012-02-161-1/+28
| | | | | | Change-Id: I8643544e8e3fd94a89d4ef49977aa00acadf18e2 Reviewed-by: Shane Kearns <ext-shane.2.kearns@nokia.com> Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* adjust license headers like in other scriptsOswald Buddenhagen2012-02-033-9/+3
| | | | | | Change-Id: I69bf891d90a5f1bffea2aee16c46e651207405e3 Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com> Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>
* Exclude .spec and .changes files from style checksSergio Ahumada2012-02-031-1/+1
| | | | | | | | Don't check for tabs in these files. Change-Id: Ide8101aade4114074293b71c6a397a25b9d2b660 Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com> Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* remove %Z from time formats - it's redundant with %cOswald Buddenhagen2012-02-021-4/+4
| | | | | | Change-Id: Idb8eb8fef7511e2bb28db9a22a43e527b5ced6b5 Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com> Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* Add missing license header.Oswald Buddenhagen2012-01-261-0/+9
| | | | | | Change-Id: I3c5f79e433631c250e6908a9617ee001cb10f65d Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com> Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* remove .pl file extension. there is no point in it being thereOswald Buddenhagen2012-01-261-0/+0
| | | | | | Change-Id: I67d240094ef78899dd37c71bc2991fefe71c530f Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com> Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* Update copyright year in license headers.Jason McDonald2012-01-052-2/+2
| | | | | Change-Id: I8d2cc9e590c54109fc86093274995f9e9508a768 Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
* don't mark Makefile.am and CMakeLists.txt as aliens any moreOswald Buddenhagen2011-12-191-1/+1
| | | | | | | | | | | it's pretty pointless to do so. we keep the exclusion of VS projects and creator (and VS) user files, though, as they clearly don't belong into a repo, and the "generated file" checker wouldn't catch them. Task-number: QTQAINFRA-425 Change-Id: I82a082923d51375270fb7ae436d94bfeb2148a9d Reviewed-by: Stephen Kelly <stephen.kelly@kdab.com>
* add sanity botOswald Buddenhagen2011-12-061-0/+195
| | | | | Change-Id: I82b781d32ac60986fee53d6a7564307ec76c6345 Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* force copy detection and a high move detection file limitOswald Buddenhagen2011-11-281-1/+1
| | | | | | to keep the noise down a bit (cherry picked from commit b46872c68681ebe4528edafc8e8df912d0d112fa)
* catch bogus author/committer emailOswald Buddenhagen2011-11-231-1/+15
| | | | (cherry picked from commit d7b51b5dcb5cb253c0fa53723dafc72a633fc09b)
* detect ELF (and PE) files as generated files by contentOswald Buddenhagen2011-11-221-2/+9
| | | | | | ELF exes typically have no extension, so they have no filename check. (cherry picked from commit 1ca6a72aaa2b1f12ef5ec4b46a3976fffdd866c9)
* remove pointless followup log message complaint suppressionOswald Buddenhagen2011-11-181-3/+0
| | | | | | | there is no reason to suppress the "2nd line not empty" message just because the summary was too long. (cherry picked from commit cf1f2aca6bee87dd979ee0da72ee1c357dc95af5)
* don't complain about long summaries in revertsOswald Buddenhagen2011-11-181-2/+2
| | | | | | | | it may in fact make sense to disable most checks for reverts. but then, there is no reason why the restored code should be just as bad as the original one, so leave them in. (cherry picked from commit 54ee24f2109e0b2b4a3ff2eee34c35b4c5663164)
* moan about reverts without justificationOswald Buddenhagen2011-11-181-8/+21
| | | | (cherry picked from commit 346c1887237ce9e44543e5bcb0b3a958b809ad81)
* suppress W-I-P check for mergesOswald Buddenhagen2011-11-181-1/+1
| | | | | | | | branch names can contain "wip", which triggers false positives. let's assume that nobody will actually keep around wip merges, so just disable the check for merges completely. (cherry picked from commit a7f118d1b92d4aa26a8e3fbe978ba75c4075835e)
* less ugly merge detection in log messageOswald Buddenhagen2011-11-181-2/+6
| | | | (cherry picked from commit d326a1cc11c5c3bed3733db85a1f52b20fee093e)
* use different message for bogus revbysOswald Buddenhagen2011-11-181-3/+4
| | | | | | people are confused if it says "wip" (cherry picked from commit 1eba816a7ba2dd1acf4dcdba722fbdfa21ee97a5)
* it's MiB, not MeBOswald Buddenhagen2011-10-241-3/+3
|
* treat qdoc files as c++Oswald Buddenhagen2011-10-111-1/+1
| | | | so it's only comments ... so what?
* treat ws-only lines as empty lines in mixws checkOswald Buddenhagen2011-10-111-4/+8
|
* special-case TAB checks for tsv filesOswald Buddenhagen2011-08-171-9/+15
| | | | we got them for squish now
* exclude /3rdparty/ from style checksOswald Buddenhagen2011-07-191-2/+3
|
* fix "generated" overrideOswald Buddenhagen2011-06-071-1/+1
|
* make pushing WIP fatal in gerrit modeOswald Buddenhagen2011-05-301-1/+1
|
* skip whitespace check for generated filesOswald Buddenhagen2011-05-301-25/+23
| | | | | | this requires that diffs are created with "infinite" context, otherwise we don't see the start of the file unless it changes. also, the regexp needs to run over all lines, not just the added ones.
* un-confuse kate's syntax highlighterOswald Buddenhagen2011-05-301-2/+2
|
* make the log message summary hint not lower the scoreOswald Buddenhagen2011-05-231-2/+2
|
* don't print override keys in gerrit modeOswald Buddenhagen2011-05-231-1/+5
|
* better error handlingOswald Buddenhagen2011-05-201-2/+12
|
* add gerrit modeOswald Buddenhagen2011-05-201-28/+22
| | | | | | - three instead of two possible exit values (ok, complain, block) - no commit header (that's implicit from the context) - no line prefixes (html frames it well enough)
* initial commitOswald Buddenhagen2011-05-052-0/+404