[jsweep] Clean validate_secrets.cjs #9225
cjs.yml
on: pull_request
js-typecheck
37s
impacted-js-tests
1m 55s
lint-js
1m 6s
Artifact API Integration
28s
Matrix: js-tests
Annotations
10 warnings
|
lint-js:
actions/setup/js/generate_history_link.cjs#L61
Wrap new URL(`${server}/search`) in try/catch — the URL constructor throws TypeError for invalid or relative URLs and will crash the action if unhandled
|
|
lint-js:
actions/setup/js/expired_entity_cleanup_helpers.cjs#L35
Prefer Number.isNaN(...) over global isNaN(...). Global isNaN() coerces non-number inputs and can hide invalid raw values
|
|
lint-js:
actions/setup/js/check_team_member.cjs#L39
core.setFailed() does not stop execution — add a control-transfer statement (for example: return, throw, break, continue, or process.exit(...)) immediately after to prevent the action from continuing in a failed state
|
|
lint-js:
actions/setup/js/artifact_client.cjs#L300
Wrap new URL(`/repos/${findBy.repositoryOwner}/${findBy.repositoryName}/actions/artifacts/${artifactId}/zip`) in try/catch — the URL constructor throws TypeError for invalid or relative URLs and will crash the action if unhandled
|
|
lint-js:
actions/setup/js/artifact_client.cjs#L252
Wrap new URL(`/repos/${findBy.repositoryOwner}/${findBy.repositoryName}/actions/runs/${findBy.workflowRunId}/artifacts`) in try/catch — the URL constructor throws TypeError for invalid or relative URLs and will crash the action if unhandled
|
|
lint-js:
actions/setup/js/artifact_client.cjs#L81
Wrap new URL(`/twirp/${TWIRP_ARTIFACT_SERVICE}/${method}`) in try/catch — the URL constructor throws TypeError for invalid or relative URLs and will crash the action if unhandled
|
|
lint-js:
actions/setup/js/artifact_client.cjs#L73
Wrap new URL(url) in try/catch — the URL constructor throws TypeError for invalid or relative URLs and will crash the action if unhandled
|
|
lint-js:
actions/setup/js/add_workflow_run_comment.cjs#L444
Avoid using an opaque whole-route string concatenation expression as the route argument of github.request(). When the entire route is dynamic, pass a typed route string from the caller instead of interpolating or concatenating a route variable
|
|
lint-js:
actions/setup/js/add_reaction_and_edit_comment.cjs#L266
Avoid using an opaque whole-route template literal with interpolations as the route argument of github.request(). When the entire route is dynamic, pass a typed route string from the caller instead of interpolating or concatenating a route variable
|
|
lint-js:
actions/setup/js/add_reaction.cjs#L193
Avoid using an opaque whole-route template literal with interpolations as the route argument of github.request(). When the entire route is dynamic, pass a typed route string from the caller instead of interpolating or concatenating a route variable
|