molecule-ai/molecule-core
41
0
Fork 2
Code Issues 103 Pull Requests 147 Actions 120 Packages Projects Releases Wiki Activity

fix(sop-checklist): move target_url assignment before N/A status post
Some checks failed
CI / Canvas Deploy Reminder (pull_request) Blocked by required conditions
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Blocked by required conditions
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Blocked by required conditions
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Blocked by required conditions
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Blocked by required conditions
Details
audit-force-merge / audit (pull_request) Has been skipped
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Failing after 13s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 27s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 15s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Failing after 24s
Details
CI / Detect changes (pull_request) Successful in 1m6s
Details
sop-tier-check / tier-check (pull_request) Successful in 26s
Details
security-review / approved (pull_request) Failing after 44s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m10s
Details
qa-review / approved (pull_request) Failing after 45s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m12s
Details
gate-check-v3 / gate-check (pull_request) Successful in 59s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 1m8s
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m22s
Details
Ops Scripts Tests / Ops scripts (unittest) (pull_request) Successful in 1m32s
Details
CI / all-required (pull_request) Successful in 3m45s
Details
CI / Python Lint & Test (pull_request) Successful in 7m32s
Details
CI / Canvas (Next.js) (pull_request) Successful in 18m38s
Details
CI / Platform (Go) (pull_request) Successful in 19m33s
Details
sop-checklist / all-items-acked (pull_request) acked: 7/7
Details

Browse Source 
NameError: target_url referenced at line ~992 before assignment at
line ~999. Fix moves the assignment before the N/A gate status post
so it is available for both status POST calls.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
Molecule AI · core-devops 2026-05-15 16:19:49 +00:00
parent 69a75f0db0
commit 71f90bba74
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
.gitea/scripts/sop-checklist.py
View File

@ -984,6 +984,8 @@ def main(argv: list[str] | None = None) -> int:
status_flag = "valid" if entry["valid"] else f"invalid: {entry['error']}"
print(f"::notice:: {g}: declared by {entry['declared_by']} — {status_flag}")
target_url = f"https://{args.gitea_host}/{args.owner}/{args.repo}/pulls/{args.pr}"
if not args.dry_run:
na_context = "sop-checklist / na-declarations (pull_request)"
client.post_status(
@ -996,8 +998,6 @@ def main(argv: list[str] | None = None) -> int:
print(f"::notice::posting status: state={state} desc={description!r}")
target_url = f"https://{args.gitea_host}/{args.owner}/{args.repo}/pulls/{args.pr}"
if args.dry_run:
print("::notice::--dry-run: not posting status")
if args.exit_on_state: