diff --git a/.github/workflows/format-crash-report-in-issue-body.yml b/.github/workflows/format-crash-report-in-issue-body.yml index 1b059bd..50e3d17 100644 --- a/.github/workflows/format-crash-report-in-issue-body.yml +++ b/.github/workflows/format-crash-report-in-issue-body.yml @@ -26,7 +26,7 @@ jobs: // Check if issue contains code block if (issueBody.includes('```')) { - console.log('Issue body seems to contain code block; skipping formatting') + core.info('Issue body seems to contain code block; skipping formatting') return } @@ -35,9 +35,9 @@ jobs: const newIssueBody = issueBody.replace(crashReportPattern, '\n```\n$&\n```\n') if (newIssueBody === issueBody) { - console.log('Did not find crash report in issue body') + core.info('Did not find crash report in issue body') } else { - console.log('Found crash report in issue body, formatting it') + core.notice('Found crash report in issue body, formatting it') github.rest.issues.update({ owner: owner, repo: repo, diff --git a/.github/workflows/minecraft-crash-reported-upstream-check.yml b/.github/workflows/minecraft-crash-reported-upstream-check.yml index 822cb17..5f4470e 100644 --- a/.github/workflows/minecraft-crash-reported-upstream-check.yml +++ b/.github/workflows/minecraft-crash-reported-upstream-check.yml @@ -24,7 +24,7 @@ jobs: const sender = context.payload.sender.login if (issue.user.login !== sender) { - console.log('Ignoring comment by user other than author') + core.info('Ignoring comment by user other than author') return } @@ -32,11 +32,11 @@ jobs: const issueLabels = issue.labels.map(label => label.name) if (issueLabels.includes(reportedUpstreamLabel)) { - console.log('Ignoring issue because it already has upstream label') + core.info('Ignoring issue because it already has upstream label') return } if (!issueLabels.includes('Minecraft')) { - console.log('Ignoring issue because it is not labeled as Minecraft') + core.info('Ignoring issue because it is not labeled as Minecraft') return } @@ -47,10 +47,10 @@ jobs: const matchedMojiraIssueKey = /(? issueBody.includes(s)) if (foundModdedStrings.length === 0) { - console.log('Did not find modded string in issue body, searching attachments') + core.info('Did not find modded string in issue body, searching attachments') // Try searching in attachments // There is currently no API so try to find URL then get attachment content, see https://github.community/t/get-files-attached-in-issue/117443 const attachmentPattern = new RegExp(`https://github\\.com/${owner}/${repo}/files/\\d+/[a-zA-Z0-9_\\-.]+`, 'g') const attachmentUrls = Array.from(issueBody.matchAll(attachmentPattern), m => m[0]) - console.log('Found attachment URLs', attachmentUrls) + core.info(`Found attachment URLs: ${attachmentUrls}`) for (const url of attachmentUrls) { let attachment = undefined try { attachment = await httpGet(url) } catch (e) { // Only log message because complete error is rather verbose - console.log('Failed getting attachment for ' + url, e.message) + core.warning(`Failed getting attachment for ${url}: ${e.message}`) continue } if (!isMinecraftIssue) { isMinecraftIssue = minecraftRegex.test(attachment) if (isMinecraftIssue) { - console.log('Found Minecraft string in attachment') + core.info('Found Minecraft string in attachment') } } @@ -87,14 +87,15 @@ jobs: let isCrashFromModdedMinecraft = foundModdedStrings.length > 0 if (isCrashFromModdedMinecraft) { - console.log('Found modded strings', foundModdedStrings) + core.notice(`Found modded strings: ${foundModdedStrings}`) } else { - console.log('Did not find modded strings') + core.info('Did not find modded strings') } isMinecraftIssue = isMinecraftIssue || isCrashFromModdedMinecraft - console.log('Is Minecraft issue: ' + isMinecraftIssue) if (isMinecraftIssue) { + core.notice('Detected issue to be about Minecraft') + let commentBody if (isCrashFromModdedMinecraft) { // Don't tell user to report modded crashes on Mojang's bug tracker; they will most likely be considered Invalid