Skip to content

refactor: comment out unused import in drop-test-tables and add biome configuration#1606

Merged
Artuomka merged 2 commits into
mainfrom
backend_biome_up
Feb 16, 2026
Merged

refactor: comment out unused import in drop-test-tables and add biome configuration#1606
Artuomka merged 2 commits into
mainfrom
backend_biome_up

Conversation

@Artuomka

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings February 16, 2026 13:33

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 performs refactoring to align the test utilities with linting standards by commenting out an unused import and adding a Biome configuration file specifically for the test directory.

Changes:

  • Commented out unused getTestKnex import in drop-test-tables.ts to fix linting warnings
  • Added backend/test/biome.json configuration extending the root config with relaxed rules for test code

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
backend/test/utils/drop-test-tables.ts Commented out unused import that was only referenced in already-commented implementation code
backend/test/biome.json New Biome linter configuration for test directory that extends root config and disables rules commonly problematic in test code (unused variables, explicit any, etc.)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Artuomka Artuomka enabled auto-merge February 16, 2026 13:44
@Artuomka Artuomka merged commit 51ec240 into main Feb 16, 2026
23 of 25 checks passed
@Artuomka Artuomka deleted the backend_biome_up branch February 16, 2026 13:46
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