summaryrefslogtreecommitdiff
path: root/Documentation/CodingGuidelines
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2010-12-13 12:22:38 -0500
committerLibravatar Junio C Hamano <gitster@pobox.com>2010-12-13 11:37:44 -0800
commitd38732c28cb43387d6d66f72bbc6fd91f3d0b516 (patch)
tree72e6896304978ce5106d77222c13b3cfad32992c /Documentation/CodingGuidelines
parentAvoid duplicate test number t7609 (diff)
downloadtgif-d38732c28cb43387d6d66f72bbc6fd91f3d0b516.tar.xz
tests: add some script lint checks
There are some common but minor errors we tend to make in writing test scripts: 1. Scripts are left non-executable. This is not usually noticed immediately because "make test" does not need the bit, but it is a matter of git policy to make them executable (and is a slight convenience when running individual scripts). 2. Two scripts are allocated the same number. Usually this happens on separate branches, and the problem only comes about during a merge. But since there is no textual conflict, the merger would have to be very observant to notice. This is also a minor error, but can make GIT_SKIP_TESTS ambiguous. This patch introduces a "test-lint" target which checks both. It is not invoked by default. You can invoke it as "make test-lint", or you can make it a prerequisite of running the tests by specifying "TEST_LINT = test-lint" in your config.mak or on the command line. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/CodingGuidelines')
0 files changed, 0 insertions, 0 deletions