INTPYTHON-380 Add linters and formatters

Author: blink1073

.git-blame-ignore-revs

@@ -0,0 +1,2 @@
+# ruff formatting of python files
+e7ea851e9b6298be479cdda6656f8c5a5dbe2614

.github/workflows/test-python.yml

@@ -17,6 +17,17 @@ env:
   MONGODB_VERSION: "7.0"
 
 jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+     - uses: actions/checkout@v4
+       with:
+          persist-credentials: false
+     - uses: actions/setup-python@v5
+     - name: "Run pre-commit"
+       run: |
+         pip install -U -q pre-commit
+         pre-commit run --all-files --hook-stage manual
   build:
     runs-on: ${{ matrix.os }}
     strategy:

.pre-commit-config.yaml

@@ -0,0 +1,49 @@
+
+repos:
+- repo: https://github.com/pre-commit/pre-commit-hooks
+  rev: v4.5.0
+  hooks:
+  - id: check-added-large-files
+  - id: check-case-conflict
+  - id: check-toml
+  - id: check-yaml
+  - id: debug-statements
+  - id: end-of-file-fixer
+  - id: forbid-new-submodules
+  - id: trailing-whitespace
+    exclude_types: [json]
+    exclude: |
+      (?x)^(.*.ambr)$
+
+# We use the Python version instead of the original version which seems to require Docker
+# https://github.com/koalaman/shellcheck-precommit
+- repo: https://github.com/shellcheck-py/shellcheck-py
+  rev: v0.9.0.6
+  hooks:
+    - id: shellcheck
+      name: shellcheck
+      args: ["--severity=warning"]
+      stages: [manual]
+
+- repo: https://github.com/sirosen/check-jsonschema
+  rev: 0.29.4
+  hooks:
+    - id: check-github-workflows
+      args: ["--verbose"]
+
+- repo: https://github.com/codespell-project/codespell
+  rev: "v2.2.6"
+  hooks:
+  - id: codespell
+    # args: ["-L", "fle,fo,infinit,isnt,nin,te,aks"]
+    stages: [manual]
+
+- repo: https://github.com/astral-sh/ruff-pre-commit
+  # Ruff version.
+  rev: v0.8.5
+  hooks:
+    # Run the linter.
+    - id: ruff
+      args: [ --fix ]
+    # Run the formatter.
+    - id: ruff-format

examples/wiki/wiki.py

@@ -30,7 +30,8 @@ def wikify(value):
     for i, part in enumerate(parts):
         if WIKIWORD.match(part):
             name = totitle(part)
-            parts[i] = "[%s](%s)" % (name, url_for("show_page", pagepath=part))
+            url = url_for("show_page", pagepath=part)
+            parts[i] = f"[{name}]({url})"
     return markdown2.markdown("".join(parts))
 
 

flask_pymongo/__init__.py

@@ -27,7 +27,6 @@
 __all__ = ("PyMongo", "ASCENDING", "DESCENDING")
 
 import hashlib
-from functools import partial
 from mimetypes import guess_type
 
 import pymongo

flask_pymongo/_version.py

@@ -1,2 +1 @@
-
-__version__ = '3.0.0.dev0'
+__version__ = "3.0.0.dev0"

flask_pymongo/helpers.py

@@ -69,7 +69,7 @@ def to_python(self, value):
         try:
             return ObjectId(value)
         except InvalidId:
-            raise abort(404)
+            raise abort(404) from None
 
     def to_url(self, value):
         return str(value)

flask_pymongo/tests/test_config.py

@@ -24,13 +24,13 @@ def doesnt_raise(exc=BaseException):
 
 class FlaskPyMongoConfigTest(FlaskRequestTest):
     def setUp(self):
-        super(FlaskPyMongoConfigTest, self).setUp()
+        super().setUp()
 
         conn = pymongo.MongoClient(port=self.port)
         conn.test.command("ping")  # wait for server
 
     def tearDown(self):
-        super(FlaskPyMongoConfigTest, self).tearDown()
+        super().tearDown()
 
         conn = pymongo.MongoClient(port=self.port)
 
@@ -87,7 +87,7 @@ class CustomDict(dict):
 
         mongo.db.things.insert_one({"_id": "thing", "val": "foo"})
 
-        assert type(mongo.db.things.find_one()) == CustomDict
+        assert mongo.db.things.find_one() is CustomDict
 
     def test_it_doesnt_connect_by_default(self):
         uri = f"mongodb://localhost:{self.port}/{self.dbname}"
@@ -112,4 +112,4 @@ def _wait_until_connected(mongo, timeout=1.0):
         if mongo.cx.nodes:
             return
         time.sleep(0.05)
-    raise CouldNotConnect("could not prove mongodb connected in %r seconds" % timeout)
+    raise CouldNotConnect(f"could not prove mongodb connected in {timeout} seconds")

flask_pymongo/tests/test_gridfs.py

@@ -18,7 +18,7 @@ def tearDown(self):
         for gridfile in files:
             gridfs.delete(gridfile._id)
 
-        super(GridFSCleanupMixin, self).tearDown()
+        super().tearDown()
 
 
 class TestSaveFile(GridFSCleanupMixin, FlaskPyMongoTest):
@@ -49,7 +49,7 @@ def test_it_returns_id(self):
 
 class TestSendFile(GridFSCleanupMixin, FlaskPyMongoTest):
     def setUp(self):
-        super(TestSendFile, self).setUp()
+        super().setUp()
 
         # make it bigger than 1 gridfs chunk
         self.myfile = BytesIO(b"a" * 500 * 1024)

flask_pymongo/tests/util.py

@@ -9,7 +9,7 @@
 
 class FlaskRequestTest(unittest.TestCase):
     def setUp(self):
-        super(FlaskRequestTest, self).setUp()
+        super().setUp()
 
         self.dbname = self.__class__.__name__
         self.app = flask.Flask("test")
@@ -18,19 +18,19 @@ def setUp(self):
         self.port = 27017
 
     def tearDown(self):
-        super(FlaskRequestTest, self).tearDown()
+        super().tearDown()
 
         self.context.pop()
 
 
 class FlaskPyMongoTest(FlaskRequestTest):
     def setUp(self):
-        super(FlaskPyMongoTest, self).setUp()
+        super().setUp()
 
         uri = f"mongodb://localhost:{self.port}/{self.dbname}"
         self.mongo = flask_pymongo.PyMongo(self.app, uri)
 
     def tearDown(self):
         self.mongo.cx.drop_database(self.dbname)
 
-        super(FlaskPyMongoTest, self).tearDown()
+        super().tearDown()