Skip to content
Node.js CI

Fix: Include the workers when npm pack #3170

Sign in to view logs

Fix: Include the workers when npm pack

Fix: Include the workers when npm pack #3170

Triggered via pull request November 8, 2023 08:31
@dennypradiptadennypradipta
opened #1167
dennypradipta:issue/1166
Status Success
Total duration 6m 40s
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
51s
lint
format
1m 18s
format
prod_test_npm
2m 59s
prod_test_npm
prod_test_binary
6m 32s
prod_test_binary
test
2m 48s
test
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
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