Fixed: Confusing Notification Message for Non-HTTP Probe #2751
haricnugrahanode.js.yml
on: pull_request
build-on-node-js-18
13m 30s
Matrix: build
Annotations
2 errors and 10 warnings
|
build (16.x, node16)
The job running on runner GitHub Actions 2 has exceeded the maximum execution time of 360 minutes.
|
|
build (16.x, node16)
The operation was canceled.
|
|
build-on-node-js-18:
packages/notification/channel/index.ts#L97
Unexpected 'todo' comment: 'TODO: add other Content type from other...'
|
|
build-on-node-js-18:
packages/notification/channel/mailgun.ts#L52
Unexpected 'todo' comment: 'TODO: Read from ENV Variables'
|
|
build-on-node-js-18:
packages/notification/channel/smtp.ts#L54
Unexpected 'todo' comment: 'TODO: Read from ENV Variables'
|
|
build-on-node-js-18:
src/components/config/index.ts#L229
Unexpected 'todo' comment: 'TODO: Assuming the first index of config...'
|
|
build-on-node-js-18:
src/components/logger/history.ts#L147
Do not use "__dirname"
|
|
build-on-node-js-18:
src/components/logger/history.ts#L486
Unexpected 'todo' comment: 'TODO: limit data stored.'
|
|
build-on-node-js-18:
src/components/logger/request-log.ts#L106
Unexpected 'todo' comment: 'TODO: make this more generic not probe...'
|
|
build-on-node-js-18:
src/jobs/tls-check.ts#L80
Unexpected 'todo' comment: 'TODO: Remove probe below'
|
|
build-on-node-js-18:
src/jobs/tls-check.ts#L93
Unexpected 'todo' comment: 'TODO: Remove validation below'
|
|
build-on-node-js-18:
src/jobs/tls-check.ts#L112
Unexpected 'todo' comment: 'TODO: invoke sendNotifications function...'
|