chore(lint): enforce explicit string concat policy

Author: hesreallyhim

dashboard_action/run.py

@@ -215,8 +215,8 @@ def _validate_secret(value: str, label: str, *, allow_weak: bool) -> None:
     if len(value) < MIN_SECRET_LENGTH and not allow_weak:
         raise ActionError(
             f"{label} is below the Reponomics dashboard secret entropy policy. "
-            "Use a generated random secret, or set allow-weak-dashboard-secret "
-            "to true if you explicitly accept the disclosure and brute-force risk."
+            + "Use a generated random secret, or set allow-weak-dashboard-secret "
+            + "to true if you explicitly accept the disclosure and brute-force risk."
         )
 
 

dashboard_action/runtime/scripts/collect.py

@@ -565,8 +565,8 @@ def resolve_repositories(
         if missing_include_only:
             print(
                 "Warning: some configured include_only repos were not eligible "
-                "for tracking (missing access, archived, forked, disabled, or "
-                "no push access): "
+                + "for tracking (missing access, archived, forked, disabled, or "
+                + "no push access): "
                 + ", ".join(missing_include_only)
             )
         resolved = [repo["full_name"] for repo in include_only_repos[:max_repos]]
@@ -588,8 +588,8 @@ def resolve_repositories(
     if missing_include:
         print(
             "Warning: some configured include repos were not eligible for "
-            "tracking (missing access, archived, forked, disabled, or no push "
-            "access): "
+            + "tracking (missing access, archived, forked, disabled, or no push "
+            + "access): "
             + ", ".join(missing_include)
         )
 
@@ -636,8 +636,8 @@ def resolve_repositories(
         print("Error: no eligible repositories found for traffic collection.")
         print(
             "Check your config or token access. Explicit includes must be "
-            "accessible, and automatic tracking excludes forks, archived repos, "
-            "disabled repos, and repos without push access."
+            + "accessible, and automatic tracking excludes forks, archived repos, "
+            + "disabled repos, and repos without push access."
         )
         sys.exit(1)
 

dashboard_action/runtime/scripts/release_notice.py

@@ -43,10 +43,10 @@ def parse_semver(value: str) -> SemVer | None:
     """Parse a constrained semver string with an optional leading v."""
     match = re.match(
         r"^\s*v?(?P<major>0|[1-9]\d*)\."
-        r"(?P<minor>0|[1-9]\d*)\."
-        r"(?P<patch>0|[1-9]\d*)"
-        r"(?:-(?P<prerelease>[0-9A-Za-z.-]+))?"
-        r"(?:\+[0-9A-Za-z.-]+)?\s*$",
+        + r"(?P<minor>0|[1-9]\d*)\."
+        + r"(?P<patch>0|[1-9]\d*)"
+        + r"(?:-(?P<prerelease>[0-9A-Za-z.-]+))?"
+        + r"(?:\+[0-9A-Za-z.-]+)?\s*$",
         value,
     )
     if not match:

pyproject.toml

@@ -43,4 +43,17 @@ line-length = 100
 target-version = "py311"
 
 [tool.ruff.lint]
-select = ["E4", "E7", "E9", "F"]
+select = ["E4", "E7", "E9", "F", "ISC"]
+
+[tool.ruff.lint.flake8-implicit-str-concat]
+allow-multiline = false
+
+[tool.ruff.lint.per-file-ignores]
+"dashboard_action/runtime/scripts/collect.py" = ["ISC002"]
+"dashboard_action/runtime/scripts/load_data.py" = ["ISC002"]
+"dashboard_action/runtime/scripts/readme_assets.py" = ["ISC002"]
+"dashboard_action/runtime/scripts/render_dashboard.py" = ["ISC002"]
+"dashboard_action/runtime/scripts/render_private_readme.py" = ["ISC002"]
+"dashboard_action/runtime/scripts/render_readme.py" = ["ISC002"]
+"dashboard_action/runtime/scripts/repo_config.py" = ["ISC002"]
+"scripts/validate_vendored_assets.py" = ["ISC002"]