Skip to content

Backend linters#1484

Merged
Artuomka merged 3 commits into
mainfrom
backend_linters
Dec 23, 2025
Merged

Backend linters#1484
Artuomka merged 3 commits into
mainfrom
backend_linters

Conversation

@Artuomka

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 22, 2025 14:58
@Artuomka Artuomka enabled auto-merge December 22, 2025 14:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment on lines +16 to +17
"start:prod": "node dist/main.js"

Copilot AI Dec 22, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
"start:prod": "node dist/main.js"
"start:prod": "node dist/main.js",

Copilot uses AI. Check for mistakes.
@Artuomka Artuomka merged commit f2cc9c4 into main Dec 23, 2025
26 of 27 checks passed
@Artuomka Artuomka deleted the backend_linters branch December 23, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants