Skip to content

Commit e7e0930

Browse files
committed
The post step should not fail if there is an error
Signed-off-by: Sora Morimoto <[email protected]>
1 parent 6b1537a commit e7e0930

File tree

13 files changed

+20
-18
lines changed

13 files changed

+20
-18
lines changed

analysis/dist/index.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/index.js

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/post/index.js

+4-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lint-doc/dist/index.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lint-fmt/dist/index.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lint-opam/dist/index.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/analysis/src/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ async function run() {
1111
process.exit(0);
1212
} catch (error) {
1313
if (error instanceof Error) {
14-
core.setFailed(error.message);
14+
core.error(error.message);
1515
}
1616
process.exit(1);
1717
}

packages/lint-doc/src/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ async function run() {
1111
process.exit(0);
1212
} catch (error) {
1313
if (error instanceof Error) {
14-
core.setFailed(error.message);
14+
core.error(error.message);
1515
}
1616
process.exit(1);
1717
}

packages/lint-fmt/src/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ async function run() {
1515
process.exit(0);
1616
} catch (error) {
1717
if (error instanceof Error) {
18-
core.setFailed(error.message);
18+
core.error(error.message);
1919
}
2020
process.exit(1);
2121
}

packages/lint-opam/src/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ async function run() {
1212
process.exit(0);
1313
} catch (error) {
1414
if (error instanceof Error) {
15-
core.setFailed(error.message);
15+
core.error(error.message);
1616
}
1717
process.exit(1);
1818
}

packages/setup-ocaml/src/cache.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ async function restoreCache(
127127
if (error instanceof Error) {
128128
core.info(error.message);
129129
}
130-
core.warning(
130+
core.notice(
131131
"An internal error has occurred in cache backend. Please check https://www.githubstatus.com for any ongoing issue in actions.",
132132
);
133133
return;
@@ -145,7 +145,7 @@ async function saveCache(key: string, paths: string[]) {
145145
if (error instanceof Error) {
146146
core.info(error.message);
147147
}
148-
core.warning(
148+
core.notice(
149149
"An internal error has occurred in cache backend. Please check https://www.githubstatus.com for any ongoing issue in actions.",
150150
);
151151
}

packages/setup-ocaml/src/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ async function run() {
88
process.exit(0);
99
} catch (error) {
1010
if (error instanceof Error) {
11-
core.setFailed(error.message);
11+
core.error(error.message);
1212
}
1313
process.exit(1);
1414
}

packages/setup-ocaml/src/post.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@ async function run() {
1515
if (error instanceof Error) {
1616
core.error(error.message);
1717
}
18-
process.exit(1);
18+
// The post step should not fail if there is an error...
19+
process.exit(0);
1920
}
2021
}
2122

0 commit comments

Comments
 (0)