Refactor: Add Base Class for Non HTTP Prober #2754
haricnugrahanode.js.yml
on: pull_request
build-on-node-js-18
14m 21s
Matrix: build
Annotations
1 error and 10 warnings
|
build (16.x, node16)
Process completed with exit code 1.
|
|
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...'
|