Skip to content

Commit 4f15e25

Browse files
committed
Avoid cluttering the workflow runs' output
When reporting anything using `core.notice()`, it will be added to the workflow runs' summary page. This can be problematic in workflows that use this here Action multiple times, e.g. in matrix builds. For example, Git's own CI run for v2.38.0 reported the same commit no less than twenty-two times: https://github.com/git/git/actions/runs/3175982844 This is excessive, and has the potential of hiding relevant, actionable notifications among a lot of clutter. Let's downgrade it to a mere info that is logged in the console, i.e. in the respective job's own log, but keep it away from the workflow runs' summary page. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent b430a04 commit 4f15e25

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/git.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ async function clone(
4242
verbose: number | boolean,
4343
cloneExtraOptions: string[] = []
4444
): Promise<void> {
45-
if (verbose) core.notice(`Cloning ${url} to ${destination}`)
45+
if (verbose) core.info(`Cloning ${url} to ${destination}`)
4646
const child = spawn(
4747
gitExePath,
4848
[
@@ -110,7 +110,7 @@ export async function getViaGit(
110110
head_sha = info.data.commit.sha
111111
}
112112
const id = `${artifactName}-${head_sha}`
113-
core.notice(`Got commit ${head_sha} for ${repo}`)
113+
core.info(`Got commit ${head_sha} for ${repo}`)
114114

115115
return {
116116
artifactName,

0 commit comments

Comments
 (0)