Skip to content
Node.js CI

Fixed: Confusing Notification Message for Non-HTTP Probe (#1072) #2755

Sign in to view logs

Fixed: Confusing Notification Message for Non-HTTP Probe (#1072)

Fixed: Confusing Notification Message for Non-HTTP Probe (#1072) #2755

Triggered via push July 13, 2023 08:16
@nicnocqueenicnocquee
pushed b88608b
main
Status Success
Total duration 15m 3s
Artifacts –

node.js.yml

on: push
build-on-node-js-18
14m 54s
build-on-node-js-18
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

20 warnings
build (16.x, node16): packages/notification/channel/index.ts#L97
Unexpected 'todo' comment: 'TODO: add other Content type from other...'
build (16.x, node16): packages/notification/channel/mailgun.ts#L52
Unexpected 'todo' comment: 'TODO: Read from ENV Variables'
build (16.x, node16): packages/notification/channel/smtp.ts#L54
Unexpected 'todo' comment: 'TODO: Read from ENV Variables'
build (16.x, node16): src/components/config/index.ts#L229
Unexpected 'todo' comment: 'TODO: Assuming the first index of config...'
build (16.x, node16): src/components/logger/history.ts#L147
Do not use "__dirname"
build (16.x, node16): src/components/logger/history.ts#L486
Unexpected 'todo' comment: 'TODO: limit data stored.'
build (16.x, node16): src/components/logger/request-log.ts#L106
Unexpected 'todo' comment: 'TODO: make this more generic not probe...'
build (16.x, node16): src/jobs/tls-check.ts#L80
Unexpected 'todo' comment: 'TODO: Remove probe below'
build (16.x, node16): src/jobs/tls-check.ts#L93
Unexpected 'todo' comment: 'TODO: Remove validation below'
build (16.x, node16): src/jobs/tls-check.ts#L112
Unexpected 'todo' comment: 'TODO: invoke sendNotifications function...'
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...'