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

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

updates:
- [github.com/Yelp/detect-secrets: v1.4.0 → v1.5.0](Yelp/detect-secrets@v1.4.0...v1.5.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.8](astral-sh/ruff-pre-commit@v0.4.1...v0.4.8)
- [github.com/rhysd/actionlint: v1.6.27 → v1.7.1](rhysd/actionlint@v1.6.27...v1.7.1)

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

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

* Address pre-commit failures

---------

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

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -23,7 +23,7 @@ repos:
       - id: yamllint
         args: [--format, parsable, -d, relaxed]
   - repo: https://github.com/Yelp/detect-secrets
-    rev: v1.4.0
+    rev: v1.5.0
     hooks:
       - id: detect-secrets
         args:
@@ -44,7 +44,7 @@ repos:
           - --exclude-files
           - ".*_test.js"
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.4.1"
+    rev: "v0.4.8"
     hooks:
       - id: ruff-format
       - id: ruff
@@ -57,7 +57,7 @@ repos:
       - id: shellcheck
         args: ["--severity=warning"]
   - repo: https://github.com/rhysd/actionlint
-    rev: v1.6.27
+    rev: v1.7.1
     hooks:
       - id: actionlint
         name: actionlint

.secrets.baseline

@@ -1,5 +1,5 @@
 {
-  "version": "1.4.0",
+  "version": "1.5.0",
   "plugins_used": [
     {
       "name": "ArtifactoryDetector"
@@ -148,7 +148,7 @@
         "filename": "docker-compose.yml",
         "hashed_secret": "965748b380ab0ab25d1846afc174a3d93a8ec06c",
         "is_verified": false,
-        "line_number": 17
+        "line_number": 8
       }
     ],
     "frontend/public/src/constants.js": [
@@ -240,5 +240,5 @@
       }
     ]
   },
-  "generated_at": "2024-05-22T12:17:57Z"
+  "generated_at": "2024-06-11T15:28:43Z"
 }

authentication/pipeline/user.py

@@ -130,7 +130,7 @@ def create_user_via_email(
                 "username": f"The username '{username}' is already taken. Please try a different username."
             },
         )
-    except Exception as exc:  # noqa: BLE001
+    except Exception as exc:
         raise UserCreationFailedException(backend, current_partial) from exc
 
     return {"is_new": True, "user": created_user, "username": created_user.username}

cms/tasks.py

@@ -40,7 +40,7 @@ def call_fastly_purge_api(relative_url):
 
     api_url = urljoin(MITX_ONLINE_FASTLY_URL, relative_url)
 
-    resp = requests.request("PURGE", api_url, headers=headers)
+    resp = requests.request("PURGE", api_url, headers=headers)  # noqa: S113
 
     if resp.status_code >= 400:  # noqa: PLR2004
         logger.error(f"Fastly API Purge call failed: {resp.status_code} {resp.reason}")  # noqa: G004

flexiblepricing/management/commands/configure_tiers.py

@@ -78,7 +78,7 @@ def handle(self, *args, **kwargs):  # pylint: disable=unused-argument  # noqa: A
                 if "course" in kwargs and kwargs["course"] is not None
                 else None
             )
-        except Exception as exc:  # noqa: BLE001
+        except Exception as exc:
             raise CommandError(
                 f"Couldn't find the course {kwargs['course']}, stopping."  # noqa: EM102
             ) from exc
@@ -89,7 +89,7 @@ def handle(self, *args, **kwargs):  # pylint: disable=unused-argument  # noqa: A
                 if "program" in kwargs and kwargs["program"] is not None
                 else None
             )
-        except Exception as exc:  # noqa: BLE001
+        except Exception as exc:
             raise CommandError(
                 f"Couldn't find the program {kwargs['program']}, stopping."  # noqa: EM102
             ) from exc

openedx/api.py

@@ -346,7 +346,7 @@ def repair_faulty_edx_user(user):
     created_user, created_auth_token = False, False
     try:
         created_user = create_edx_user(user)
-    except Exception as e:  # noqa: BLE001
+    except Exception as e:
         # 409 means we have a username conflict - pass in that case so we can
         # try to create the api auth tokens; re-raise otherwise
 
@@ -672,7 +672,7 @@ def enroll_in_edx_course_runs(
             results.append(enrollment)
         except HTTPError as exc:  # noqa: PERF203
             raise EdxApiEnrollErrorException(user, course_run, exc) from exc
-        except Exception as exc:  # pylint: disable=broad-except  # noqa: BLE001
+        except Exception as exc:  # pylint: disable=broad-except
             raise UnknownEdxApiEnrollException(user, course_run, exc) from exc
     return results
 
@@ -848,7 +848,7 @@ def subscribe_to_edx_course_emails(user, course_run):
         result = edx_client.email_settings.subscribe(course_run.courseware_id)
     except HTTPError as exc:
         raise EdxApiEmailSettingsErrorException(user, course_run, exc) from exc
-    except Exception as exc:  # noqa: BLE001
+    except Exception as exc:
         raise UnknownEdxApiEmailSettingsException(user, course_run, exc) from exc
     return result
 
@@ -874,7 +874,7 @@ def unsubscribe_from_edx_course_emails(user, course_run):
         result = edx_client.email_settings.unsubscribe(course_run.courseware_id)
     except HTTPError as exc:
         raise EdxApiEmailSettingsErrorException(user, course_run, exc) from exc
-    except Exception as exc:  # noqa: BLE001
+    except Exception as exc:
         raise UnknownEdxApiEmailSettingsException(user, course_run, exc) from exc
     return result