Skip to content

chore(ci): use check-runs API for npm-publish CI gate#46

Merged
StefanSteiner merged 1 commit into
tableau:mainfrom
StefanSteiner:ssteiner/fix-npm-publish-ci-check
May 27, 2026
Merged

chore(ci): use check-runs API for npm-publish CI gate#46
StefanSteiner merged 1 commit into
tableau:mainfrom
StefanSteiner:ssteiner/fix-npm-publish-ci-check

Conversation

@StefanSteiner
Copy link
Copy Markdown
Contributor

The previous polling logic queried the legacy commit-status API (/commits/{sha}/status), which only returns checks reported via the older Status API. GitHub Actions reports results as check-runs, not statuses, so the poll returned state=pending forever even after CI fully succeeded. Switch to the check-runs API and aggregate by required check name, treating success/skipped/neutral as passing.

The previous polling logic queried the legacy commit-status API
(/commits/{sha}/status), which only returns checks reported via the
older Status API. GitHub Actions reports results as check-runs, not
statuses, so the poll returned state=pending forever even after CI
fully succeeded. Switch to the check-runs API and aggregate by
required check name, treating success/skipped/neutral as passing.
@StefanSteiner StefanSteiner merged commit f0a8f85 into tableau:main May 27, 2026
10 checks 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