Skip to content

ci: remove redundant qualification result job - #111

Merged
JJLiebig merged 1 commit into
mainfrom
ci/remove-ci-result-job
Oct 29, 2026
Merged

JJLiebig merged 1 commit into
mainfrom
ci/remove-ci-result-job

Conversation

@JJLiebig

@JJLiebig JJLiebig commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Qualification currently allocates a second runner only to repeat the profile check and the correctness job's result. Remove that serial tail and report the selected profile, exact validated commit, and status in the existing correctness job's log and step summary.

The existing full/deploy validator, SHA output, quality/type/package/browser checks, evidence uploads, and publish dependency remain intact. Main has no branch protection or effective repository rules requiring the removed aggregate check.

Validation:

  • Native pre-commit passed: formatting, full lint, supply-chain watchlist, and four TypeScript checks.
  • Temporary focused checks passed: YAML parsing; full/deploy accepted and invalid profile rejected; summary success/failure/cancellation output; preserved SHA output and publish qualification dependency.
  • Fast Review Suite rvw_9d457888: dual signoff, no findings, review_green passed on 2b12eff87c146aff9e4586b0d180b96829d1b34c.
  • Hosted Qualification passed: Quality, typecheck, packed package/browser suite (57 tests / 12 files), evidence preservation, and the retained summary. The summary reports the PR merge commit and Correctness=success.
  • The deploy profile was checked locally at its profile/qualification boundary; no release or publish was dispatched.

No package behavior changes or deployment.

@JJLiebig
JJLiebig merged commit 98ab54a into main Oct 29, 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