[tor-commits] [tor/master] Update CodingStandards.md for changes-file stuff

nickm at torproject.org nickm at torproject.org
Thu Sep 21 20:40:31 UTC 2017


commit 2032d7ca6f0d082c3774c922944feb7d73c8a851
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Sep 21 16:39:49 2017 -0400

    Update CodingStandards.md for changes-file stuff
    
    'check-changes' is now part of "make check", and must pass.
---
 doc/HACKING/CodingStandards.md | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/doc/HACKING/CodingStandards.md b/doc/HACKING/CodingStandards.md
index 1012a4c9c..2980b775f 100644
--- a/doc/HACKING/CodingStandards.md
+++ b/doc/HACKING/CodingStandards.md
@@ -96,17 +96,14 @@ you can use `git describe --contains <sha1 of commit>`.
 If at all possible, try to create this file in the same commit where you are
 making the change.  Please give it a distinctive name that no other branch will
 use for the lifetime of your change. To verify the format of the changes file,
-you can use `make check-changes`.
+you can use `make check-changes`.  This is run automatically as part of
+`make check` -- if it fails, we must fix it before we release.  These
+checks are implemented in `scripts/maint/lintChanges.py`.
 
 When we go to make a release, we will concatenate all the entries
 in changes to make a draft changelog, and clear the directory. We'll
 then edit the draft changelog into a nice readable format.
 
-To make sure that stuff is in the right format, we use
-scripts/maint/lintChanges.py to check the changes files for
-(superficial) validity.  You can run this script on your own changes
-files!
-
 What needs a changes file?
 
    * A not-exhaustive list: Anything that might change user-visible



More information about the tor-commits mailing list