feat: auto-setup GitHub webhook for PR Review workflows #975
alari76Annotations
22 warnings
|
test-and-lint (20)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
test-and-lint (20):
server/claude-process.ts#L429
Unsafe assignment of an `any` value
|
|
test-and-lint (20):
server/claude-process.ts#L382
Forbidden non-null assertion
|
|
test-and-lint (20):
server/claude-process.ts#L369
Unnecessary conditional, value is always falsy
|
|
test-and-lint (20):
server/claude-process.ts#L354
Unnecessary optional chain on a non-nullish value
|
|
test-and-lint (20):
server/claude-process.ts#L297
Unsafe assignment of an `any` value
|
|
test-and-lint (20):
server/claude-process.ts#L255
Forbidden non-null assertion
|
|
test-and-lint (20):
server/claude-process.ts#L253
Returning a void expression from an arrow function shorthand is forbidden. Please add braces to the arrow function
|
|
test-and-lint (20):
server/claude-process.ts#L252
Forbidden non-null assertion
|
|
test-and-lint (20):
server/auth-routes.ts#L24
Unnecessary conditional, value is always truthy
|
|
test-and-lint (20):
server/approval-manager.ts#L310
Prefer the safe `: unknown` for a `catch` callback variable
|
|
test-and-lint (22)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
test-and-lint (22):
server/claude-process.ts#L429
Unsafe assignment of an `any` value
|
|
test-and-lint (22):
server/claude-process.ts#L382
Forbidden non-null assertion
|
|
test-and-lint (22):
server/claude-process.ts#L369
Unnecessary conditional, value is always falsy
|
|
test-and-lint (22):
server/claude-process.ts#L354
Unnecessary optional chain on a non-nullish value
|
|
test-and-lint (22):
server/claude-process.ts#L297
Unsafe assignment of an `any` value
|
|
test-and-lint (22):
server/claude-process.ts#L255
Forbidden non-null assertion
|
|
test-and-lint (22):
server/claude-process.ts#L253
Returning a void expression from an arrow function shorthand is forbidden. Please add braces to the arrow function
|
|
test-and-lint (22):
server/claude-process.ts#L252
Forbidden non-null assertion
|
|
test-and-lint (22):
server/auth-routes.ts#L24
Unnecessary conditional, value is always truthy
|
|
test-and-lint (22):
server/approval-manager.ts#L310
Prefer the safe `: unknown` for a `catch` callback variable
|