Skip to content

Biome#22

Merged
benhalverson merged 6 commits intomainfrom
biome
Oct 28, 2025
Merged

Biome#22
benhalverson merged 6 commits intomainfrom
biome

Conversation

@benhalverson
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings October 27, 2025 23:09
Copy link
Contributor

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 project from ESLint to Biome for linting and formatting, with comprehensive code style updates applied throughout the codebase.

  • Replaced ESLint configuration and dependencies with Biome
  • Applied automated formatting changes (quote style, indentation, bracket placement)
  • Added type safety improvements and accessibility enhancements

Reviewed Changes

Copilot reviewed 38 out of 42 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
package.json Removed ESLint dependencies, added Biome, updated lint/format scripts
biome.json Added new Biome configuration with formatting and linting rules
.eslintrc.cjs Removed ESLint configuration file
src/config.ts Added runtime validation for environment variables using Zod
src/main.tsx Added root element existence check before rendering
src/pages/*.tsx Applied formatting changes and added missing button types, accessibility attributes
src/components/*.tsx Applied formatting changes, added accessibility improvements (title attributes, htmlFor)
src/context/*.tsx Applied formatting and added biome-ignore comments for exhaustive deps
src/utils/webauthn.ts Applied quote style formatting
src/interfaces/*.ts Standardized import syntax with type imports
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

Copilot AI review requested due to automatic review settings October 27, 2025 23:15
Copy link
Contributor

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

Copilot reviewed 40 out of 45 changed files in this pull request and generated 6 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

Copilot AI review requested due to automatic review settings October 28, 2025 07:16
@benhalverson benhalverson marked this pull request as ready for review October 28, 2025 07:18
Copy link
Contributor

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

Copilot reviewed 44 out of 49 changed files in this pull request and generated 3 comments.


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

@benhalverson benhalverson merged commit aca8bcb into main Oct 28, 2025
1 check passed
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

Comments