Skip to content

Commit 676d651

Browse files
authored
Merge pull request #227 from proto-kit/feature/ci-loglevel
Override loglevel to error for unit tests
2 parents c0b98ce + d208d1e commit 676d651

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

.github/workflows/pull-request-develop.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,8 @@ jobs:
9191

9292
- name: "Test"
9393
run: npm run test:ci
94+
env:
95+
IN_CI: true
9496

9597
integration:
9698
runs-on: ubuntu-latest

packages/common/src/log.ts

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,11 @@ function logProvable(
2525
}
2626
/* eslint-enable */
2727

28+
// eslint-disable-next-line @typescript-eslint/strict-boolean-expressions
29+
if (process.env?.IN_CI ?? false) {
30+
loglevel.setLevel("ERROR");
31+
}
32+
2833
const timeMap: Record<string, number> = {};
2934

3035
function time(label = "time") {
@@ -111,7 +116,10 @@ export const log = {
111116
},
112117

113118
setLevel: (level: LogLevelDesc) => {
114-
loglevel.setLevel(level);
119+
// eslint-disable-next-line @typescript-eslint/strict-boolean-expressions
120+
if (!(process.env?.IN_CI ?? false)) {
121+
loglevel.setLevel(level);
122+
}
115123
},
116124

117125
get levels() {

0 commit comments

Comments
 (0)