Skip to content

Commit

Permalink
Clarify info message
Browse files Browse the repository at this point in the history
  • Loading branch information
pllim committed Jan 4, 2021
1 parent 163017d commit 55d986f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ function run() {
core.info(` ${accepted_flags[i]}`);
}
if (accepted_flags.some(v => msg.includes(v))) {
core.setFailed(`"${commit.data.message}" contains directive to skip CI, failing this check`);
core.setFailed(`"${commit.data.message}" contains directive to skip, failing this check`);
/* Instead of failing, can also try to cancel but the token needs write access,
so we cannot implement this for OSS in reality. */
/*
Expand All @@ -76,7 +76,7 @@ function run() {
*/
}
else {
core.info(`No directive to skip CI found in "${commit.data.message}", moving on...`);
core.info(`No directive to skip found in "${commit.data.message}", moving on...`);
}
}
catch (err) {
Expand Down
4 changes: 2 additions & 2 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ async function run() {
}

if (accepted_flags.some(v => msg.includes(v))) {
core.setFailed(`"${commit.data.message}" contains directive to skip CI, failing this check`);
core.setFailed(`"${commit.data.message}" contains directive to skip, failing this check`);

/* Instead of failing, can also try to cancel but the token needs write access,
so we cannot implement this for OSS in reality. */
Expand All @@ -42,7 +42,7 @@ async function run() {
});
*/
} else {
core.info(`No directive to skip CI found in "${commit.data.message}", moving on...`);
core.info(`No directive to skip found in "${commit.data.message}", moving on...`);
}
} catch(err) {
core.setFailed(`Action failed with error ${err}`);
Expand Down

0 comments on commit 55d986f

Please sign in to comment.