ci: strip fork CI to CEF-only — remove upstream actions + #code-review QA (re-target to main) - #6
Merged
Merged
Conversation
Remove 19 workflows that belong to the upstream OSS project and are noise (or would run red) on the CEF fork: benchmarks (5), tracer demo, terraform bench, openclaw e2e, hermes tests, k8s-alert trigger, windows labels, community bots (automerge, celebrate, good-first-issue, greptile), the Homebrew/PyPI release + docker-publish pipelines, closed-loop reminder, and the interactive-shell live test. Keep: ci.yml, codeql.yml, synthetic-deterministic.yml (core CI/security/ regression) + the CEF QA workflows (hiring-coach-qa, qa-slack-poller, ux-login-probe, telegram-preflight). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PkNM3j2VZmxijQsWFZMPiy
Removes the Slack-#code-review 'run QA on Stage' machinery that ran on every deploy/merge signal: - .github/workflows/qa-slack-poller.yml (cron */10 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. The scheduled post-publish QA (qa/quality/ux/dashboard jobs, twice-daily cron) is unchanged. Feature-2 PR-blame (correlate / pr-context) is not carried over. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PkNM3j2VZmxijQsWFZMPiy
Greptile code reviewThis repo uses Greptile for automated review. Before merge, aim for Confidence Score: 5/5 with zero unresolved review threads — see CONTRIBUTING.md. Run a review — add a PR comment with: Give it ~5-10 minutes (sometimes longer) for results, then fix feedback and re-trigger until you reach Confidence Score: 5/5. Optional: automate with the greploop skill. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Re-do of #4 against the correct base. #4 was stacked on the sync branch and got merged into that branch, not
main, so its removals never reachedmain. Same two commits, now targetingmain:qa-slack-poller.yml,qa_slack_poller.py,qa_deploy_summary.py, and the deploy-QA parts ofhiring-coach-qa.yml).Diff is exactly those two removal commits (main already has the sync via #3).
🤖 Generated with Claude Code