Merge pull request #285 from mxmeinhold/pylint-fixes

Author: mxmeinhold

.github/workflows/python-app.yml

@@ -31,7 +31,7 @@ jobs:
           if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
       - name: Lint with pylint
         run: |
-          pylint packet
+          pylint packet/routes packet
 
   typecheck:
     runs-on: ubuntu-latest

packet/__init__.py

@@ -78,6 +78,7 @@
 app.logger.info('OIDCAuth configured')
 
 # Sentry
+# pylint: disable=abstract-class-instantiated
 sentry_sdk.init(
     dsn=app.config['SENTRY_DSN'],
     integrations=[FlaskIntegration(), SqlalchemyIntegration()]

packet/routes/api.py

@@ -210,8 +210,8 @@ def upperclassman_stats(uid):
 
 @app.route('/readiness')
 def readiness() -> tuple[str, int]:
-    '''A basic healthcheck. Returns 200 to indicate flask is running'''
-    return "ready", 200
+    """A basic healthcheck. Returns 200 to indicate flask is running"""
+    return 'ready', 200
 
 
 def commit_sig(packet, was_100, uid):