[pre-commit.ci] pre-commit autoupdate (#167)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/adrienverge/yamllint.git: v1.35.1 → v1.37.0](https://github.com/adrienverge/yamllint.git/compare/v1.35.1...v1.37.0)
- [github.com/astral-sh/ruff-pre-commit: v0.5.7 → v0.11.7](astral-sh/ruff-pre-commit@v0.5.7...v0.11.7)
- [github.com/rhysd/actionlint: v1.7.1 → v1.7.7](rhysd/actionlint@v1.7.1...v1.7.7)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/adrienverge/yamllint.git: v1.35.1 → v1.37.1](https://github.com/adrienverge/yamllint.git/compare/v1.35.1...v1.37.1)
- [github.com/astral-sh/ruff-pre-commit: v0.5.7 → v0.11.8](astral-sh/ruff-pre-commit@v0.5.7...v0.11.8)
- [github.com/rhysd/actionlint: v1.7.1 → v1.7.7](rhysd/actionlint@v1.7.1...v1.7.7)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* checks

* items

* change logs

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: CP <[email protected]>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -3,7 +3,7 @@
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v4.6.0
+  rev: v5.0.0
   hooks:
   - id: trailing-whitespace
   - id: end-of-file-fixer
@@ -27,7 +27,7 @@ repos:
     - '80'
     - --preserve-quotes
 - repo: https://github.com/adrienverge/yamllint.git
-  rev: v1.35.1
+  rev: v1.37.1
   hooks:
   - id: yamllint
     args: [--format, parsable, -d, relaxed]
@@ -41,7 +41,7 @@ repos:
     additional_dependencies: ["gibberish_detector"]
 - repo: https://github.com/astral-sh/ruff-pre-commit
   # Ruff version.
-  rev: 'v0.5.7'
+  rev: 'v0.11.8'
   hooks:
   - id: ruff-format
   - id: ruff
@@ -65,7 +65,7 @@ repos:
   hooks:
   - id: shellcheck
 - repo: https://github.com/rhysd/actionlint
-  rev: v1.7.1
+  rev: v1.7.7
   hooks:
   - id: actionlint
     name: actionlint

conftest.py

@@ -13,7 +13,7 @@
 
 
 @pytest.fixture(scope="session")
-def django_db_modify_db_settings_pants_suffix() -> None:  # noqa: PT004
+def django_db_modify_db_settings_pants_suffix() -> None:
     skip_if_no_django()
 
     slot_id = environ.get("PANTS_EXECUTION_SLOT", None)
@@ -23,13 +23,13 @@ def django_db_modify_db_settings_pants_suffix() -> None:  # noqa: PT004
 
 
 @pytest.fixture(scope="session")
-def django_db_modify_db_settings_parallel_suffix(  # noqa: PT004
+def django_db_modify_db_settings_parallel_suffix(
     django_db_modify_db_settings_pants_suffix,  # noqa: ARG001
 ) -> None:
     skip_if_no_django()
 
 
-@pytest.fixture()
+@pytest.fixture
 def learner_drf_client(learner):
     """DRF API test client that is authenticated with the user"""
     # import is here to avoid trying to load django before settings are initialized
@@ -40,7 +40,7 @@ def learner_drf_client(learner):
     return client
 
 
-@pytest.fixture()
+@pytest.fixture
 def learner(db):  # noqa: ARG001
     """Fixture for a default learner"""
     # import is here to avoid trying to load django before settings are initialized
@@ -49,7 +49,7 @@ def learner(db):  # noqa: ARG001
     return UserFactory.create()
 
 
-@pytest.fixture()
+@pytest.fixture
 def learner_and_oauth2(learner):
     """Fixture for a default learner and oauth2 records"""
     # import is here to avoid trying to load django before settings are initialized
@@ -75,31 +75,31 @@ def learner_and_oauth2(learner):
     )
 
 
-@pytest.fixture()
+@pytest.fixture
 def staff_user(db):  # noqa: ARG001
     """Staff user fixture"""
     from mitol.common.factories import UserFactory
 
     return UserFactory.create(is_staff=True)
 
 
-@pytest.fixture()
+@pytest.fixture
 def user_client(learner):
     """Django test client that is authenticated with the user"""
     client = Client()
     client.force_login(learner)
     return client
 
 
-@pytest.fixture()
+@pytest.fixture
 def staff_client(staff_user):
     """Django test client that is authenticated with the staff user"""
     client = Client()
     client.force_login(staff_user)
     return client
 
 
-@pytest.fixture()
+@pytest.fixture
 def google_sheets_base_settings(settings):
     """Fixture for base google sheets settings"""
     settings.MITOL_GOOGLE_SHEETS_ENROLLMENT_CHANGE_SHEET_ID = "1"
@@ -109,14 +109,14 @@ def google_sheets_base_settings(settings):
     return settings
 
 
-@pytest.fixture()
+@pytest.fixture
 def google_sheets_service_creds_settings(settings):
     """Fixture for google sheets settings configured for a service account"""
     settings.MITOL_GOOGLE_SHEETS_DRIVE_SERVICE_ACCOUNT_CREDS = '{"credentials": "json"}'
     return settings
 
 
-@pytest.fixture()
+@pytest.fixture
 def google_sheets_client_creds_settings(settings):
     """Fixture gor google sheets settings configured with OAuth"""
     settings.MITOL_GOOGLE_SHEETS_DRIVE_CLIENT_ID = "nhijg1i.apps.googleusercontent.com"

src/common/changelog.d/20250506_102349_collinp_pre_commit_ci_update_config.md

@@ -0,0 +1,42 @@
+<!--
+A new scriv changelog fragment.
+
+Uncomment the section that is right (remove the HTML comment wrapper).
+For top level release notes, leave all the headers commented out.
+-->
+
+<!--
+### Removed
+
+- A bullet item for the Removed category.
+
+-->
+<!--
+### Added
+
+- A bullet item for the Added category.
+
+-->
+<!--
+### Changed
+
+- pre-commit auto update fixes.
+-->
+<!--
+### Deprecated
+
+- A bullet item for the Deprecated category.
+
+-->
+<!--
+### Fixed
+
+- A bullet item for the Fixed category.
+
+-->
+<!--
+### Security
+
+- A bullet item for the Security category.
+
+-->

src/common/mitol/common/models.py

@@ -32,7 +32,7 @@ def update(self, **kwargs):
         Automatically update updated_on timestamp when .update(). This is because .update()
         does not go through .save(), thus will not auto_now, because it happens on the
         database level without loading objects into memory.
-        """  # noqa: E501, D402
+        """  # noqa: E501
         if "updated_on" not in kwargs:
             kwargs["updated_on"] = now_in_utc()
         return super().update(**kwargs)

src/common/mitol/common/templatetags/render_bundle.py

@@ -60,7 +60,7 @@ def render_bundle(
         return _render_tags(bundle, added_attrs)
     except OSError:
         # webpack-stats.json doesn't exist
-        return mark_safe("")  # noqa: S308
+        return mark_safe("")
 
 
 def _render_tags(bundle: Iterator[dict], added_attrs: str = "") -> SafeText:

src/google_sheets_deferrals/changelog.d/20250506_102412_collinp_pre_commit_ci_update_config.md

@@ -0,0 +1,43 @@
+<!--
+A new scriv changelog fragment.
+
+Uncomment the section that is right (remove the HTML comment wrapper).
+For top level release notes, leave all the headers commented out.
+-->
+
+<!--
+### Removed
+
+- A bullet item for the Removed category.
+
+-->
+<!--
+### Added
+
+- A bullet item for the Added category.
+
+-->
+<!--
+### Changed
+
+- pre-commit auto update fixes.
+
+-->
+<!--
+### Deprecated
+
+- A bullet item for the Deprecated category.
+
+-->
+<!--
+### Fixed
+
+- A bullet item for the Fixed category.
+
+-->
+<!--
+### Security
+
+- A bullet item for the Security category.
+
+-->

src/google_sheets_deferrals/mitol/google_sheets_deferrals/management/commands/process_deferral_requests.py

@@ -24,7 +24,7 @@ def handle(self, *args, **options):  # noqa: ARG002
         deferral_request_handler = DeferralRequestHandler()
         self.stdout.write("Handling refunds and updating spreadsheet...")
         results = deferral_request_handler.process_sheet(
-            limit_row_index=options.get("row", None)
+            limit_row_index=options.get("row")
         )
         self.stdout.write(
             self.style.SUCCESS(f"Deferral sheet successfully processed.\n{results}")

src/google_sheets_refunds/changelog.d/20250506_102419_collinp_pre_commit_ci_update_config.md

@@ -0,0 +1,43 @@
+<!--
+A new scriv changelog fragment.
+
+Uncomment the section that is right (remove the HTML comment wrapper).
+For top level release notes, leave all the headers commented out.
+-->
+
+<!--
+### Removed
+
+- A bullet item for the Removed category.
+
+-->
+<!--
+### Added
+
+- A bullet item for the Added category.
+
+-->
+<!--
+### Changed
+
+- pre-commit auto update fixes.
+
+-->
+<!--
+### Deprecated
+
+- A bullet item for the Deprecated category.
+
+-->
+<!--
+### Fixed
+
+- A bullet item for the Fixed category.
+
+-->
+<!--
+### Security
+
+- A bullet item for the Security category.
+
+-->

src/google_sheets_refunds/mitol/google_sheets_refunds/management/commands/process_refund_requests.py

@@ -24,7 +24,7 @@ def handle(self, *args, **options):  # noqa: ARG002
         refund_request_handler = RefundRequestHandler()
         self.stdout.write("Handling refunds and updating spreadsheet...")
         results = refund_request_handler.process_sheet(
-            limit_row_index=options.get("row", None)
+            limit_row_index=options.get("row")
         )
         self.stdout.write(
             self.style.SUCCESS(f"Refund sheet successfully processed.\n{results}")

src/payment_gateway/changelog.d/20250506_102427_collinp_pre_commit_ci_update_config.md

@@ -0,0 +1,43 @@
+<!--
+A new scriv changelog fragment.
+
+Uncomment the section that is right (remove the HTML comment wrapper).
+For top level release notes, leave all the headers commented out.
+-->
+
+<!--
+### Removed
+
+- A bullet item for the Removed category.
+
+-->
+<!--
+### Added
+
+- A bullet item for the Added category.
+
+-->
+<!--
+### Changed
+
+- pre-commit auto update fixes.
+
+-->
+<!--
+### Deprecated
+
+- A bullet item for the Deprecated category.
+
+-->
+<!--
+### Fixed
+
+- A bullet item for the Fixed category.
+
+-->
+<!--
+### Security
+
+- A bullet item for the Security category.
+
+-->