[extensions-web] review: Auto-approve more basic files



commit 8d5a41f874adfeca665d9ea2062b1d8d5d4bfe92
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Mon Apr 2 20:36:10 2012 -0400

    review: Auto-approve more basic files
    
    These should be safe to approve. We're using name, not filename in
    case someone wants to check-in a 'README.md' file or something.

 sweettooth/review/views.py |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
---
diff --git a/sweettooth/review/views.py b/sweettooth/review/views.py
index 79af605..d94951e 100644
--- a/sweettooth/review/views.py
+++ b/sweettooth/review/views.py
@@ -376,6 +376,10 @@ def safe_to_auto_approve(changes):
 
         name, ext = os.path.splitext(filename)
 
+        # Harmless common metadata files.
+        if name in ['README', 'CHANGELOG', 'COPYING', 'LICENSE']:
+            continue
+
         # Translations and stylesheet updates are safe.
         if ext in ['.mo', '.po', '.css']:
             continue



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]