Skip to content

feat: async mode + PR head sha for backend-posted checks#38

Merged
finalerock44 merged 2 commits into
mainfrom
feat/async-backend-checks
Jul 3, 2026
Merged

feat: async mode + PR head sha for backend-posted checks#38
finalerock44 merged 2 commits into
mainfrom
feat/async-backend-checks

Conversation

@finalerock44

@finalerock44 finalerock44 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor
  • Respect async: when set, submit the run and exit 0 without polling (saves CI minutes). The DeviceCloud GitHub App reports the result as a check on the commit/PR when the run completes.
  • Fix gh_sha to use the PR head sha (pr.head.sha) instead of the throwaway merge commit so the check lands on the developer-visible commit.
  • Rebuild dist, document async + the GitHub App in the README, add a params test for the PR head-sha behaviour.

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

- Respect `async`: when set, submit the run and exit 0 without polling (saves CI
  minutes). The DeviceCloud GitHub App reports the result as a check on the
  commit/PR when the run completes.
- Fix gh_sha to use the PR head sha (pr.head.sha) instead of the throwaway merge
  commit so the check lands on the developer-visible commit.
- Rebuild dist, document async + the GitHub App in the README, add a params test
  for the PR head-sha behaviour.
@finalerock44 finalerock44 self-assigned this Jun 30, 2026
@finalerock44 finalerock44 merged commit 63453cf into main Jul 3, 2026
2 checks passed
@finalerock44 finalerock44 deleted the feat/async-backend-checks branch July 3, 2026 16:02
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