Skip to content
Node.js CI

Fix: Validation on Alert Assertion #3201

Sign in to view logs

Fix: Validation on Alert Assertion

Fix: Validation on Alert Assertion #3201

Re-run triggered November 15, 2023 02:59
@haricnugrahaharicnugraha
#1179
haricnugraha:fix/1177-validate-alert-assertion
Status Failure
Total duration 2m 58s
Artifacts –
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

node.js.yml

on: pull_request
lint
57s
lint
format
46s
format
prod_test_npm
2m 30s
prod_test_npm
prod_test_binary
6m 31s
prod_test_binary
test
2m 47s
test
Fit to window
Zoom out
Zoom in

Annotations

1 error and 10 warnings
test
Process completed with exit code 1.
lint: packages/notification/channel/__test__/pagerduty.test.ts#L45
Unexpected any. Specify a different type
lint: packages/notification/channel/__test__/pagerduty.test.ts#L160
Expected property shorthand
lint: packages/notification/channel/desktop.ts#L79
Missing braces in case clause
lint: packages/notification/channel/desktop.ts#L92
Prefer `String#codePointAt()` over `String#charCodeAt()`
lint: packages/notification/channel/dingtalk.ts#L132
Missing braces in case clause
lint: packages/notification/channel/googlechat.ts#L103
Missing braces in case clause
lint: packages/notification/channel/googlechat.ts#L137
Missing braces in case clause
lint: packages/notification/channel/googlechat.ts#L175
Missing braces in case clause
lint: packages/notification/channel/googlechat.ts#L213
Missing braces in case clause
lint: packages/notification/channel/googlechat.ts#L271
Missing braces in case clause