commit 570f7a2318efe40f9f8aff19a03a674b9e1ae4e9
author Gilles Peskine <Gilles.Peskine@arm.com> Thu Jul 04 19:31:33 2019 +0200
committer Gilles Peskine <Gilles.Peskine@arm.com> Fri Jul 05 10:56:19 2019 +0200
tree e168dfab03806a0c6ae517c6fcaff029c1b9a9f9
parent 081daf0dd3a86ce7f9052c6136aa2db9c32de36d

Allow TODO in code

Don't reject TODO in code. Fix #2587

tests/scripts/check-files.py

diff --git a/tests/scripts/check-files.py b/tests/scripts/check-files.py
index a5b5950..e2d3f4a 100755
--- a/tests/scripts/check-files.py
+++ b/tests/scripts/check-files.py
@@ -6,7 +6,7 @@
 """
 This script checks the current state of the source code for minor issues,
 including incorrect file permissions, presence of tabs, non-Unix line endings,
-trailing whitespace, presence of UTF-8 BOM, and TODO comments.
+trailing whitespace, and presence of UTF-8 BOM.
 Note: requires python 3, must be run from Mbed TLS root.
 """
 
@@ -168,19 +168,6 @@
             return True
         return False
 
-class TodoIssueTracker(LineIssueTracker):
-    """Track lines containing ``TODO``."""
-
-    heading = "TODO present:"
-    files_exemptions = frozenset([
-        os.path.basename(__file__),
-        "benchmark.c",
-        "pull_request_template.md",
-    ])
-
-    def issue_with_line(self, line, _filepath):
-        return b"todo" in line.lower()
-
 
 class IntegrityChecker(object):
     """Sanity-check files under the current directory."""
@@ -210,7 +197,6 @@
             TrailingWhitespaceIssueTracker(),
             TabIssueTracker(),
             MergeArtifactIssueTracker(),
-            TodoIssueTracker(),
         ]
 
     @staticmethod