/checkpatch

Checkpatch for Tarantool

Primary LanguagePerlGNU General Public License v2.0GPL-2.0

Checkpatch for Tarantool

This repository contains the checkpatch.pl script, which is used for checking patches submitted for the Tarantool project against the Tarantool contributor's guide.

The checkpatch.pl script was forked from the checkpatch.pl script used for checking patches submitted for the Linux kernel.

The documentation is here.

The GitHub action we use to automatically check Tarantool commits when a pull request is created is here.

If you find any bugs, please don't hesitate to report them to the issue tracker.

Quick start

To check all Git commits on the current branch, run the following command in your local Tarantool Git directory:

git clone git@github.com:tarantool/checkpatch.git
checkpatch/checkpatch.pl -g master..HEAD

An error message reported by checkpatch looks like this:

ERROR: trailing whitespace
#41: FILE: changelogs/unreleased/gh-7207-backtrace-perf-degrade.md:3:
+* Fixed performance degrade of fiber backtrace collection $

To get more detailed error messages, pass the -v (--verbose) flag. It will augment each error message with an extract from the documentation:

ERROR: trailing whitespace
#41: FILE: changelogs/unreleased/gh-7207-backtrace-perf-degrade.md:3:
+* Fixed performance degrade of fiber backtrace collection $

Trailing whitespace should always be removed.
Some editors highlight the trailing whitespace and cause visual
distractions when editing files.

If you don't want to clutter the checkpatch output, you can instead pass the --show-types flag. It will make checkpatch print the type of each reported error, which you can then use to look up the full error description in the documentation. This is how our GitHub action works. For example,

ERROR:TRAILING_WHITESPACE: trailing whitespace
#41: FILE: changelogs/unreleased/gh-7207-backtrace-perf-degrade.md:3:
+* Fixed performance degrade of fiber backtrace collection $