Improve Actions docs and comment accuracy (follow-up) (#756)

Signed-off-by: Glenn Jocher <[email protected]>

.github/workflows/cla.yml

@@ -37,7 +37,7 @@ jobs:
           path-to-document: "https://docs.ultralytics.com/help/CLA" # CLA document
           # Branch must not be protected
           branch: cla-signatures
-          allowlist: dependabot[bot],github-actions,[pre-commit*,pre-commit*,bot*
+          allowlist: dependabot[bot],github-actions,pre-commit*,bot*
 
           remote-organization-name: ultralytics
           remote-repository-name: cla

actions/__init__.py

@@ -29,4 +29,4 @@
 #     ├── test_summarize_pr.py
 #     └── ...
 
-__version__ = "0.2.14"
+__version__ = "0.2.15"

actions/review_pr.py

@@ -430,7 +430,7 @@ def post_review_summary(event: Action, review_data: dict, review_number: int) ->
     comments = review_data.get("comments", [])
     summary = review_data.get("summary") or ""
 
-    # Don't approve if error occurred, inline comments exist, or critical/high severity issues
+    # Don't approve if error occurred, inline comments exist, or medium-or-higher severity issues
     has_error = not summary or ERROR_MARKER in summary
     has_inline_comments = review_data.get("comments_before_filtering", 0) > 0
     has_issues = any(c.get("severity") not in ["LOW", "SUGGESTION", None] for c in comments)

actions/scan_prs.py

@@ -267,7 +267,7 @@ def run():
                 total_skipped += 1
 
     summary.append(f"\n**Summary:** Found {total_found} | Merged {total_merged} | Skipped {total_skipped}")
-    print(f"\n📊 Dependabot Summary: Found {total_found} | Merged {total_merged} | Skipped {total_skipped}")
+    print(f"\n📊 Auto-Merge Summary: Found {total_found} | Merged {total_merged} | Skipped {total_skipped}")
 
     if summary_file := os.getenv("GITHUB_STEP_SUMMARY"):
         with open(summary_file, "a") as f:

actions/update_file_headers.py

@@ -176,7 +176,7 @@ def update_file(file_path, prefix, block_start, block_end, base_header):
 
 
 def main(*args, **kwargs):
-    """Automates file header updates for all files in the specified directory."""
+    """Automates file header updates for supported file types under the current working directory."""
     event = Action(*args, **kwargs)
     current_year = datetime.now().year
     repository = (event.repository or "").lower()

actions/utils/common_utils.py

@@ -11,7 +11,7 @@ from urllib import parse
 
 import requests
 
-# Common directories to exclude when traversing file trees (used by docstring formatter, header updater, etc.)
+# Common directories to exclude when traversing file trees (used by the Python docstring formatter)
 COMMON_EXCLUDED_DIRS = frozenset(
     {
         ".git",
@@ -274,7 +274,7 @@ def clean_url(url):
 
 
 def allow_redirect(start="", end=""):
-    """Check if URL should be skipped based on simple rules."""
+    """Check if a redirect target should be applied based on simple allow rules."""
     start_lower = start.lower()
     end_lower = end.lower()
     return (

actions/utils/openai_utils.py

@@ -159,7 +159,7 @@ def _get_default_model() -> str:
 
 
 def get_review_model() -> str:
-    """Get model for PR reviews, using REVIEW_MODEL if set, otherwise default model."""
+    """Get model for PR reviews, using REVIEW_MODEL if set, otherwise PR_REVIEW_MODEL_DEFAULT."""
     return REVIEW_MODEL or PR_REVIEW_MODEL_DEFAULT
 
 

cleanup-disk/README.md

@@ -2,7 +2,7 @@
 
 # 🧹 Disk Space Cleanup Action
 
-Cleans up disk space on GitHub Actions runners by removing unnecessary tool caches and swap space. Frees up ~19GB total space.
+Cleans up disk space on Ubuntu GitHub Actions runners by removing unnecessary tool caches and swap space. Frees up ~19GB total space.
 
 ## 🚀 Usage
 

retry/README.md

@@ -56,7 +56,7 @@ steps:
 | --------------------- | ------------------------------------------------------------ | -------- | ------------- |
 | `run`                 | Command to run                                               | Yes      | -             |
 | `retries`             | Number of retry attempts after initial run                   | No       | `3`           |
-| `timeout_minutes`     | Maximum total time in minutes for all attempts combined      | No       | `360`         |
+| `timeout_minutes`     | Maximum total time in minutes, checked between attempts      | No       | `360`         |
 | `retry_delay_seconds` | Base delay between retries in seconds                        | No       | `10`          |
 | `backoff`             | Backoff strategy: `exponential` (base \* 2^n) or `fixed`     | No       | `exponential` |
 | `jitter`              | Randomize delay to 80-120% of value to avoid thundering herd | No       | `true`        |

retry/action.yml

@@ -26,7 +26,7 @@ name: "Step-Level Retry"
 description: "Retries a step while preserving its full context"
 inputs:
   timeout_minutes:
-    description: "Maximum total time in minutes for all attempts"
+    description: "Maximum total time in minutes for all attempts (checked between attempts; does not interrupt a running attempt)"
     required: false
     default: "360"
   retries: