Skip to content

fix(ci): configure downstream QPK pin sync identity#217

Merged
Pigbibi merged 1 commit into
mainfrom
codex/fix-downstream-pin-sync-identity-20260710
Jul 10, 2026
Merged

fix(ci): configure downstream QPK pin sync identity#217
Pigbibi merged 1 commit into
mainfrom
codex/fix-downstream-pin-sync-identity-20260710

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Fix the QPK downstream pin-sync workflow failure: cloned downstream repositories had no Git commit identity, so the fanout job could not create pin-update PRs.

The script now configures a repository-local bot name/email before committing. Optional environment overrides are non-secret.

Validation:

  • python3 -m compileall -q scripts/open_downstream_qpk_pin_prs.py
  • python3 -m ruff check scripts/open_downstream_qpk_pin_prs.py
  • python3 -m pytest -q (517 passed, 1 skipped)
  • git diff --check.

Co-Authored-By: Codex <noreply@openai.com>
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

Merge allowed: No issues found


Review by Codex PR Review bot • PR

@Pigbibi

Pigbibi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Retriggering the organization Codex gate with the one-minute wait setting.

@Pigbibi Pigbibi closed this Jul 10, 2026
@Pigbibi Pigbibi reopened this Jul 10, 2026
@Pigbibi

Pigbibi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Retrying once after the VPS health inspection confirmed the audit service is active; the prior review failure was a Cloudflare 522 origin timeout.

@Pigbibi Pigbibi closed this Jul 10, 2026
@Pigbibi Pigbibi reopened this Jul 10, 2026
@Pigbibi Pigbibi merged commit acdf954 into main Jul 10, 2026
7 of 9 checks passed
@Pigbibi Pigbibi deleted the codex/fix-downstream-pin-sync-identity-20260710 branch July 10, 2026 12:50
Pigbibi added a commit that referenced this pull request Jul 10, 2026
Co-authored-by: Codex <noreply@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant