chore: add Biome code quality checks - #64
Closed
nihmadev wants to merge 4 commits into
Closed
Annotations
2 errors and 11 warnings
|
Check pull request changes
Process completed with exit code 1.
|
|
Check pull request changes:
app/main/helpers/requests.js#L1
Formatter would have printed the following content:
|
|
Complete job
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Check pull request changes:
app/auth.js#L306
This async function lacks an await expression.
|
|
Check pull request changes:
app/auth.js#L260
Function has 5 parameters, but only 4 are allowed.
|
|
Check pull request changes:
app/auth.js#L182
Magic number detected. Extract it to a constant with a meaningful name.
|
|
Check pull request changes:
app/auth.js#L178
Magic number detected. Extract it to a constant with a meaningful name.
|
|
Check pull request changes:
app/auth.js#L178
Block statements are preferred in this position.
|
|
Check pull request changes:
app/auth.js#L3
Using Node.js modules is forbidden.
|
|
Check pull request changes:
app/auth.js#L3
A Node.js builtin module should be imported with the node: protocol.
|
|
Check pull request changes:
app/auth.js#L2
Using Node.js modules is forbidden.
|
|
Check pull request changes:
app/auth.js#L2
A Node.js builtin module should be imported with the node: protocol.
|
|
Check pull request changes:
app/auth.js#L2
Use ESM imports instead of require.
|
background
wait
wait-all
cancel
parallel
Loading