Skip to content

chore(ci): dedupe QA live credential check#97947

Merged
RomneyDa merged 1 commit into
mainfrom
codex/dedupe-qa-live-credential-check
Jun 29, 2026
Merged

chore(ci): dedupe QA live credential check#97947
RomneyDa merged 1 commit into
mainfrom
codex/dedupe-qa-live-credential-check

Conversation

@RomneyDa

Copy link
Copy Markdown
Member

What Problem This Solves

Resolves a CI maintenance problem where the same QA live credential-validation step was copied across four Convex-backed transport jobs in .github/workflows/qa-live-transports-convex.yml.

Why This Change Was Made

This uses a YAML anchor on the first validation step and aliases the other three exact copies. Before editing, the four candidate blocks had one parsed YAML value and one normalized-source hash, so only fully identical chunks were deduplicated.

This PR was prepared with Codex.

User Impact

No product behavior changes. CI maintainers get one canonical QA live credential-validation block for these transport jobs.

Evidence

Duplicate proof before editing:

  • Jobs checked: run_live_telegram, run_live_discord, run_live_whatsapp, run_live_slack
  • Normalized source hash for all four: 3c43c9b9b63080cc205e46a9cf8398cccc3e60a8312c728cd52aa2b6d1f85123
  • Parsed YAML values: one unique value across all four blocks

Validation:

  • actionlint .github/workflows/qa-live-transports-convex.yml
  • git diff --check

@openclaw-barnacle openclaw-barnacle Bot added size: S maintainer Maintainer-authored PR labels Jun 29, 2026
@clawsweeper

clawsweeper Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: chore(ci): dedupe QA live credential check This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@RomneyDa
RomneyDa merged commit a107b95 into main Jun 29, 2026
95 of 97 checks passed
@RomneyDa
RomneyDa deleted the codex/dedupe-qa-live-credential-check branch June 29, 2026 23:23
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jun 30, 2026
QiuYuang pushed a commit to QiuYuang/openclaw that referenced this pull request Jul 1, 2026
chenyangjun-xy pushed a commit to chenyangjun-xy/openclaw that referenced this pull request Jul 1, 2026
chenyangjun-xy pushed a commit to chenyangjun-xy/openclaw that referenced this pull request Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant