thebadking/obscura
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #656 from h4ckf0r0day/ci/fix-pr-check-scope

Browse Source 
ci: stop enforcing PR description headings
This commit is contained in:
SGavrl
2026-08-14 10:48:38 +00:00
committed by GitHub
parent 923bee0053 d9a86479ac
commit 9ac65da7d4
1 changed files with 0 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
scripts/ci/pr_policy.py
View File

@@ -4,8 +4,6 @@
from __future__ import annotations from __future__ import annotations
import argparse import argparse
import json
import os
import re import re
import subprocess import subprocess
import sys import sys
@@ -17,7 +15,6 @@ USES_RE = re.compile(r"^\s*-?\s*uses\s*:\s*(.+?)\s*$")
FULL_ACTION_REF_RE = re.compile(r"^[^@]+@[0-9a-f]{40}$") FULL_ACTION_REF_RE = re.compile(r"^[^@]+@[0-9a-f]{40}$")
PINNED_DOCKER_REF_RE = re.compile(r"^docker://[^@]+@sha256:[0-9a-f]{64}$") PINNED_DOCKER_REF_RE = re.compile(r"^docker://[^@]+@sha256:[0-9a-f]{64}$")
MAX_CONTROL_FILE_BYTES = 2 * 1024 * 1024 MAX_CONTROL_FILE_BYTES = 2 * 1024 * 1024
REQUIRED_SECTIONS = ("What changed", "Validation", "Rendering", "Performance", "Checklist")
ALWAYS_BLOCKED_WORKFLOW_ADDITIONS = ( ALWAYS_BLOCKED_WORKFLOW_ADDITIONS = (
"pull_request_target", "pull_request_target",
"workflow_run", "workflow_run",
@@ -60,30 +57,6 @@ def changed_files(base: str, head: str) -> list[str]:
return [part.decode("utf-8", "surrogateescape") for part in raw.split(b"\0") if part] return [part.decode("utf-8", "surrogateescape") for part in raw.split(b"\0") if part]
def event_body() -> str | None:
if os.environ.get("GITHUB_EVENT_NAME") != "pull_request":
return None
event_path = os.environ.get("GITHUB_EVENT_PATH")
if not event_path:
return ""
try:
event = json.loads(Path(event_path).read_text(encoding="utf-8"))
except (OSError, json.JSONDecodeError):
return ""
body = event.get("pull_request", {}).get("body")
return body if isinstance(body, str) else ""
def check_template(errors: list[str]) -> None:
body = event_body()
if body is None:
return
for heading in REQUIRED_SECTIONS:
pattern = re.compile(rf"(?mi)^##\s+{re.escape(heading)}\s*$")
if not pattern.search(body):
errors.append(f"PR description is missing the '## {heading}' section")
def check_artifacts(files: list[str], errors: list[str], warnings: list[str]) -> None: def check_artifacts(files: list[str], errors: list[str], warnings: list[str]) -> None:
for name in files: for name in files:
path = PurePosixPath(name) path = PurePosixPath(name)
@@ -232,7 +205,6 @@ def main() -> None:
files = changed_files(args.base, args.head) files = changed_files(args.base, args.head)
errors: list[str] = [] errors: list[str] = []
warnings: list[str] = [] warnings: list[str] = []
check_template(errors)
check_artifacts(files, errors, warnings) check_artifacts(files, errors, warnings)
check_control_symlinks(files, errors) check_control_symlinks(files, errors)
check_new_vendor(args.base, files, errors) check_new_vendor(args.base, files, errors)