New configuration option - Allowing "pylint:" commments

docs/usage/configuration.md

@@ -52,6 +52,17 @@ import python_ta
 python_ta.check_all(..., load_default_config=False)
 ```
 
+## Allowing 'pylint:' Comments
+
+PythonTA allows you to choose whether you want to locally disable checks by using 'pylint:' in a comment, i.e. it
+lets you choose whether or not you want to allow comments that begin with 'pylint:'. The default value for this option is False, i.e. PythonTA by default would not allow you to use 'pylint:' in a comment.
+
+```python
+import python_ta
+
+python_ta.check_all(..., config = {"allow-pylint-comments" : True})
+```
+
 ## Custom Error Messages
 
 By default, PythonTA overwrites some of pylint's error messages with its own to make them more beginner-friendly.

python_ta/__init__.py

@@ -140,7 +140,8 @@ def _check(
             errs = []  # Errors caught in files for data submission
             config = {}  # Configuration settings for data submission
             for file_py in get_file_paths(locations):
-                if not _verify_pre_check(file_py):
+                allowed_pylint = linter.config.allow_pylint_comments
+                if not _verify_pre_check(file_py, allowed_pylint):
                     continue  # Check the other files
                 # Load config file in user location. Construct new linter each
                 # time, so config options don't bleed to unintended files.
@@ -289,6 +290,15 @@ def reset_linter(
                 "help": "Path to patch config toml file.",
             },
         ),
+        (
+            "allow-pylint-comments",
+            {
+                "default": False,
+                "type": "yn",
+                "metavar": "<yn>",
+                "help": "allows or disallows pylint: comments",
+            },
+        ),
         (
             "use-pyta-error-messages",
             {
@@ -358,13 +368,15 @@ def get_file_paths(rel_path: AnyStr) -> Generator[AnyStr, None, None]:
                 yield os.path.join(root, filename)  # Format path, from root.
 
 
-def _verify_pre_check(filepath: AnyStr) -> bool:
+def _verify_pre_check(filepath: AnyStr, allowed_pylint: bool) -> bool:
     """Check student code for certain issues."""
     # Make sure the program doesn't crash for students.
     # Could use some improvement for better logging and error reporting.
     try:
-        # Check for inline "pylint:" comment, which may indicate a student
-        # trying to disable a check.
+        if allowed_pylint:
+            return True
+            # Check for inline "pylint:" comment, which may indicate a student
+            # trying to disable a check.
         with tokenize.open(os.path.expanduser(filepath)) as f:
             for tok_type, content, _, _, _ in tokenize.generate_tokens(f.readline):
                 if tok_type != tokenize.COMMENT:

python_ta/allow_pylint_comments_false_testcase.py

@@ -0,0 +1,14 @@
+def some_function():
+    x = 5  # pylint: disable something
+    y = 0
+    result = x / y
+    return result
+
+
+a = 10
+b = 20
+if __name__ == "__main__":
+    import python_ta
+
+    options = {"allow-pylint-comments": False}
+    python_ta.check_all(config=options)

python_ta/allow_pylint_comments_true_testcase.py

@@ -0,0 +1,14 @@
+def some_function():
+    x = 5  # pylint: disable something
+    y = 0
+    result = x / y
+    return result
+
+
+a = 10
+b = 20
+if __name__ == "__main__":
+    import python_ta
+
+    options = {"allow-pylint-comments": True}
+    python_ta.check_all(config=options)

python_ta/config/.pylintrc

@@ -23,6 +23,9 @@ pyta-server-address = http://127.0.0.1:5000
 # Path to messages_config toml file
 # messages-config-path = messages.toml
 
+# Allowing pylint: comments
+allow-pylint-comments = no
+
 # Set whether the default error messages by pylint should be overwritten by PythonTA's custom messages
 use-pyta-error-messages = yes