Added docstrings

Author: Penaz91

pylint_django/checkers/modelfilter_forloop.py

@@ -6,6 +6,9 @@
 
 
 class ModelFilterForLoopChecker(checkers.BaseChecker):
+    """
+    Checks for usage of "Model.manager.filter() inside of for loops
+    """
     __implements__ = (interfaces.IAstroidChecker)
 
     name = "model-filter-forloop-checker"
@@ -14,9 +17,9 @@ class ModelFilterForLoopChecker(checkers.BaseChecker):
         f"R{BASE_ID}06": (
             "Consider using '__in' queries",
             "consider-using-in-queries",
-            "Using 'Model.filter()' or 'Model.get() inside a for loop may "
-            "impact performance. Consider using a single query with as '__in' "
-            "filter instead, outside of the loop."
+            "Using 'Model.manager.filter()' or 'Model.manager.get() inside a "
+            "for loop may negatively impact performance. Consider using a "
+            "single query with as '__in' filter instead, outside of the loop."
         ),
     }
 

pylint_django/checkers/modelsave_forloop.py

@@ -7,6 +7,10 @@
 
 
 class ModelSaveForLoopChecker(checkers.BaseChecker):
+    """
+    Checks for usage of Model.manager.create() or Model.save() inside of for
+    loops
+    """
     __implements__ = (interfaces.IAstroidChecker)
 
     name = "model-save-forloop-checker"
@@ -15,15 +19,16 @@ class ModelSaveForLoopChecker(checkers.BaseChecker):
         f"R{BASE_ID}04": (
             "Consider using 'Model.bulk_create()'",
             "consider-using-bulk-create",
-            "Using 'Model.create()' inside a for loop may impact performance. "
-            "Consider using 'Model.bulk_create()' instead."
+            "Using 'Model.manager.create()' inside a for loop may negatively "
+            "impact performance. Consider using 'Model.manager.bulk_create()' "
+            "instead."
         ),
         f"R{BASE_ID}05": (
             "Consider using 'Model.bulk_*()",
             "consider-using-bulk-create-save",
-            "Using 'Model.save()' inside a for loop may impact performance. "
-            "Consider using 'Model.bulk_update()' or "
-            "'Model.bulk_create()' instead."
+            "Using 'Model.save()' inside a for loop may negatively impact "
+            "performance. Consider using 'Model.manager.bulk_update()' or "
+            "'Model.manager.bulk_create()' instead."
         ),
     }
 

pylint_django/checkers/queryset_iterator.py

@@ -7,6 +7,10 @@
 
 
 class QuerysetIteratorForLoopChecker(checkers.BaseChecker):
+    """
+    Checks for usage of "QuerySet.all()" in the head of a for loop,
+    eventually suggesting the usage of ".iterator()"
+    """
     __implements__ = (interfaces.IAstroidChecker)
 
     name = "queryset-iterator-forloop-checker"