diff options
| author | Joe Gordon <jogo@cloudscaling.com> | 2012-08-02 14:36:58 -0700 |
|---|---|---|
| committer | Joe Gordon <jogo@cloudscaling.com> | 2012-08-08 14:53:38 -0700 |
| commit | 47a1bbe3114c4e5bb84c5b050ecc9baff83375cc (patch) | |
| tree | 8b1f368a4b4fc9173346be5ee79f2db097cbdb99 /tools | |
| parent | 28a03d9aded97732038eadd5bbcb21282f1b246a (diff) | |
| download | nova-47a1bbe3114c4e5bb84c5b050ecc9baff83375cc.tar.gz nova-47a1bbe3114c4e5bb84c5b050ecc9baff83375cc.tar.xz nova-47a1bbe3114c4e5bb84c5b050ecc9baff83375cc.zip | |
Add a 50 char git title limit test to hacking.
* add N802 to hacking.py
Change-Id: I8262531b4b3f8f3a1a2a37679904cf4864cae7b6
Diffstat (limited to 'tools')
| -rwxr-xr-x | tools/hacking.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tools/hacking.py b/tools/hacking.py index edb06525e..4eb2141f7 100755 --- a/tools/hacking.py +++ b/tools/hacking.py @@ -431,6 +431,7 @@ def once_git_check_commit_title(): nova HACKING recommends not referencing a bug or blueprint in first line, it should provide an accurate description of the change N801 + N802 Title limited to 50 chars """ #Get title of most recent commit @@ -453,6 +454,9 @@ def once_git_check_commit_title(): print ("N801: git commit title ('%s') should provide an accurate " "description of the change, not just a reference to a bug " "or blueprint" % title.strip()) + if len(title.decode('utf-8')) > 72: + print ("N802: git commit title ('%s') should be under 50 chars" + % title.strip()) if __name__ == "__main__": #include nova path |
