[#8539] add some pylint checks

Allura/allura/command/base.py

@@ -87,7 +87,7 @@ def config(self):
         return tg.config
 
     def basic_setup(self):
-        global log, M
+        global log, M  # noqa: PLW0603
         if self.args[0]:
             # Probably being called from the command line - load the config
             # file

Allura/allura/lib/helpers.py

@@ -32,7 +32,7 @@
 import logging
 import string
 import random
-import pickle as pickle
+import pickle
 from hashlib import sha1
 from datetime import datetime, timedelta
 from collections import defaultdict, OrderedDict

Allura/allura/lib/patches.py

@@ -29,7 +29,7 @@
 
 _patched = False
 def apply():
-    global _patched
+    global _patched  # noqa: PLW0603
     if _patched:
         return
     _patched = True

Allura/allura/tests/functional/test_discuss.py

@@ -426,8 +426,7 @@ def attach_link(self):
             if 'attachment' in alink['href']:
                 alink = str(alink['href'])
                 return alink
-        else:
-            assert False, 'attachment link not found'
+        assert False, 'attachment link not found'
 
     def test_attach(self):
         r = self.app.post(self.post_link + 'attach',

Allura/allura/websetup/bootstrap.py

@@ -56,7 +56,7 @@ def bootstrap(command, conf, vars):
         REGISTRY.register(ew.widget_context,
                           ew.core.WidgetContext('http', ew.ResourceManager()))
 
-    create_test_data = asbool(os.getenv('ALLURA_TEST_DATA', True))
+    create_test_data = asbool(os.getenv('ALLURA_TEST_DATA', 'true'))
 
     # if this is a test_run, skip user project creation to save time
     make_user_projects = not test_run

ruff.toml

@@ -33,6 +33,9 @@ lint.select = [
     "G010", # logging.warn
     "T10",  # debugger breakpoints
     "T20",  # print()
+    "PLC",
+    "PLE",
+    "PLW",
     "FA",   # future annotations (to ensure compatibility with `target-version`)
 ]
 
@@ -57,6 +60,7 @@ lint.ignore = [
     'S324', # md5 & sha1
     'S603', # subprocess
     'S607', # partial path
+    'PLW2901', # loop var overwritten
 ]
 
 [lint.per-file-ignores]

scripts/ApacheAccessHandler.py

@@ -52,7 +52,7 @@ def load_requests_lib(req):
             exec(compile(open(activate_this, "rb").read(), activate_this, 'exec'), {'__file__': activate_this})  # noqa: S102
         except Exception as e:
             log(req, "Couldn't activate venv via {}: {}".format(activate_this, repr(e)))
-    global requests
+    global requests  # noqa: PLW0603
     import requests as requests_lib
     requests = requests_lib