Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

chore: use @stegripe/eslint-config #11

chore: use @stegripe/eslint-config

chore: use @stegripe/eslint-config #11

Triggered via pull request July 21, 2024 07:47
Status Failure
Total duration 27s
Artifacts

test.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
test / lint / lint: index.js#L1
`./yt-dlp-utils/index.js` import should occur after import of `prism-media`
test / lint / lint: index.js#L2
`node:fs` import should occur after import of `node:child_process`
test / lint / lint: index.js#L4
`node:path` import should occur after import of `node:module`
test / lint / lint: index.js#L9
Unexpected use of 'process'. Import process from `node:process` instead
test / lint / lint: index.js#L9
Unexpected use of the global variable 'process'. Use 'require("process")' instead
test / lint / lint: index.js#L39
Unexpected use of 'process'. Import process from `node:process` instead
test / lint / lint: index.js#L39
Unexpected use of the global variable 'process'. Use 'require("process")' instead
test / lint / lint: index.js#L41
Unexpected sync method: 'existsSync'
test / lint / lint: index.js#L42
Unexpected sync method: 'rmSync'
test / lint / lint: index.js#L49
Unexpected sync method: 'execSync'
test / lint / lint: index.js#L4
Use default import for module `node:path`
test / lint / lint: index.js#L68
Remove unused catch binding `err`
test / lint / lint: index.js#L68
The catch parameter `err` should be named `error`
test / lint / lint: src/bot.ts#L22
Prefer top-level await over using a promise chain
test / lint / lint: src/bot.ts#L22
The catch parameter `e` should be named `error`
test / lint / lint: src/bot.ts#L22
Identifier name 'e' is too short (< 2)
test / lint / lint: src/commands/developers/EvalCommand.ts#L25
/```(?:[^\s]+\n)?(.*?)\n?```/gs can be optimized to /```(?:\S+\n)?(.*?)\n?```/gs