Skip to content

Fix biome lint errors in configuration#37

Merged
Dhravya merged 1 commit intomainfrom
claude/fix-biome-lint-errors-LOIQF
Jan 3, 2026
Merged

Fix biome lint errors in configuration#37
Dhravya merged 1 commit intomainfrom
claude/fix-biome-lint-errors-LOIQF

Conversation

@Dhravya
Copy link
Member

@Dhravya Dhravya commented Jan 3, 2026

  • Update biome.json schema version to 2.3.11
  • Fix biome.json configuration (include -> includes, remove invalid keys)
  • Add node: protocol to child_process import
  • Replace string concatenation with template literals
  • Fix biome-ignore comment placeholders
  • Add biome-ignore for intentional any type usage
  • Remove non-null assertions with proper null checks
  • Change let to const where appropriate
  • Convert T[] array syntax to Array for consistency
  • Refactor assignment in while expression to separate statements

- Update biome.json schema version to 2.3.11
- Fix biome.json configuration (include -> includes, remove invalid keys)
- Add node: protocol to child_process import
- Replace string concatenation with template literals
- Fix biome-ignore comment placeholders
- Add biome-ignore for intentional any type usage
- Remove non-null assertions with proper null checks
- Change let to const where appropriate
- Convert T[] array syntax to Array<T> for consistency
- Refactor assignment in while expression to separate statements
@Dhravya Dhravya merged commit 3ec6426 into main Jan 3, 2026
1 check passed
@claude
Copy link
Contributor

claude bot commented Jan 3, 2026

Claude finished @Dhravya's task —— View job


Reviewing PR #37: Biome Lint Error Fixes

Review Checklist:

  • Examine biome.json configuration changes
  • Review bin/run.ts modifications
  • Analyze src/client-config.ts updates
  • Check src/commands/install.ts changes
  • Review src/detect-transport.ts modifications
  • Assess code quality and best practices
  • Evaluate potential bugs or issues
  • Consider performance implications
  • Check security concerns
  • Assess test coverage impact

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