Skip to content

ci: restore deploy-triggered QA (#code-review) to last-working state - #8

Merged
jaxter03 merged 1 commit into
mainfrom
restore/deploy-qa-workflows
Aug 3, 2026
Merged

ci: restore deploy-triggered QA (#code-review) to last-working state#8
jaxter03 merged 1 commit into
mainfrom
restore/deploy-qa-workflows

Conversation

@jaxter03

@jaxter03 jaxter03 commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

What

Restores the deploy-triggered / #code-review QA machinery that PR #6 removed (commit aa01090c), bringing it back to its last-working snapshot — commit 58687708, the parent of that removal.

Restored files, byte-identical to 58687708:

  • .github/workflows/qa-slack-poller.yml — cron poller reading #code-review
  • .github/scripts/qa_slack_poller.py
  • .github/scripts/qa_deploy_summary.py
  • .github/workflows/hiring-coach-qa.yml — the deploy-QA parts: workflow_call entry, the prs / component / slack_thread_ts / slack_channel inputs, and the deploy_summary job

Scope / safety

  • The scheduled post-publish QA (qa/quality/ux/dashboard, twice-daily cron) is untouched — it was never removed.
  • hiring-coach-qa.yml on main differed from the working snapshot only by the 81-line deploy block (0 other changes), so this restore loses nothing else.
  • YAML validated.

Heads-up

Brings the deploy-QA machinery back to its last-working state (commit 5868770,
the parent of the removal aa01090 that landed via PR #6):

- .github/workflows/qa-slack-poller.yml (cron reading #code-review)
- .github/scripts/qa_slack_poller.py, qa_deploy_summary.py
- the deploy-QA parts of hiring-coach-qa.yml: the workflow_call entry, the
  prs/component/slack_thread_ts/slack_channel inputs, and the deploy_summary job

Files are byte-identical to 5868770; the scheduled post-publish QA is
unaffected.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jaxter03
jaxter03 merged commit 850517e into main Aug 3, 2026
1 check passed
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