Skip to content

Commit

Permalink
- include conclusion when updating the check runs
Browse files Browse the repository at this point in the history
  • Loading branch information
mikepenz committed Nov 21, 2021
1 parent 9aa13a6 commit 8f57b37
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 9 deletions.
8 changes: 3 additions & 5 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,11 +55,10 @@ export async function run(): Promise<void> {
foundResults && testResult.annotations.length === 0
? 'success'
: 'failure'
const status: 'completed' = 'completed'
const head_sha =
commit || (pullRequest && pullRequest.head.sha) || github.context.sha
core.info(
`ℹ️ Posting status '${status}' with conclusion '${conclusion}' to ${link} (sha: ${head_sha})`
`ℹ️ Posting with conclusion '${conclusion}' to ${link} (sha: ${head_sha})`
)

core.endGroup()
Expand Down Expand Up @@ -88,6 +87,7 @@ export async function run(): Promise<void> {
output: {
title,
summary,
conclusion,
annotations: testResult.annotations.slice(0, 50)
}
}
Expand All @@ -100,7 +100,7 @@ export async function run(): Promise<void> {
...github.context.repo,
name: checkName,
head_sha,
status,
status: 'completed',
conclusion,
output: {
title,
Expand Down

0 comments on commit 8f57b37

Please sign in to comment.