Skip to content

Conversation

@Artuomka
Copy link
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
Copy link

Copilot AI left a comment

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"

Copy link

Copilot AI Dec 22, 2025

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