Skip to content

Commit

Permalink
Merge pull request #642 from mikepenz/fix/640
Browse files Browse the repository at this point in the history
Fix catching the error for using tokens with insufficient permission scope
  • Loading branch information
mikepenz authored Aug 20, 2022
2 parents 44dce0f + fa61d94 commit e187db4
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 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.

4 changes: 2 additions & 2 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ export async function run(): Promise<void> {

try {
for (const testResult of testResults) {
annotateTestResult(testResult, token, headSha, annotateOnly, updateCheck, annotateNotice)
await annotateTestResult(testResult, token, headSha, annotateOnly, updateCheck, annotateNotice)
}
} catch (error) {
core.error(`❌ Failed to create checks using the provided token. (${error})`)
Expand All @@ -111,7 +111,7 @@ export async function run(): Promise<void> {
const supportsJobSummary = process.env['GITHUB_STEP_SUMMARY']
if (jobSummary && supportsJobSummary) {
try {
attachSummary(testResults, detailedSummary)
await attachSummary(testResults, detailedSummary)
} catch (error) {
core.error(`❌ Failed to set the summary using the provided token. (${error})`)
}
Expand Down

0 comments on commit e187db4

Please sign in to comment.