Backend linters#1484
Conversation
There was a problem hiding this comment.
Pull request overview
This PR migrates the backend linting setup from ESLint/Prettier to Biome by removing ESLint and Prettier dependencies and configurations across multiple packages (backend, shared-code, and rocketadmin-agent), while updating the Biome configuration with new formatting rules.
- Removed ESLint and Prettier dependencies and their associated configuration files
- Updated biome.json with lineWidth setting and changed quoteStyle from double to single quotes
- Applied formatting changes to code (import reordering, indentation changes to tabs)
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Removed all ESLint, Prettier, and related plugin dependencies from the lockfile |
| shared-code/package.json | Removed lint script and ESLint/Prettier dev dependencies |
| shared-code/eslint.config.mjs | Deleted ESLint configuration file |
| rocketadmin-agent/package.json | Removed format and lint scripts and ESLint/Prettier dev dependencies; contains JSON syntax error |
| rocketadmin-agent/eslint.config.mjs | Deleted ESLint configuration file |
| backend/package.json | Removed format script and ESLint/Prettier dev dependencies |
| backend/eslint.config.mjs | Deleted ESLint configuration file |
| biome.json | Added lineWidth: 120 and changed quoteStyle from "double" to "single" |
| backend/src/app.controller.ts | Applied new formatting with tabs and reordered imports |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "start:prod": "node dist/main.js" | ||
|
|
There was a problem hiding this comment.
There is a trailing comma missing after the "start:prod" script entry. Line 16 should end with a comma since it's followed by another property in the JSON object. Additionally, line 17 contains unnecessary whitespace before the closing brace.
| "start:prod": "node dist/main.js" | |
| "start:prod": "node dist/main.js", |
No description provided.