Skip to content

Conversation

nikgraf
Copy link
Collaborator

@nikgraf nikgraf commented Jul 22, 2025

did the lint:fix command and interestingly it renamed bunch of unsed variables e.g. https://github.com/graphprotocol/hypergraph/pull/358/files#diff-31393e44cb529ff5a5c7860accdf17cb98b94ff38f42405c70d94988310618b5R330

and it’s a lot more “aggressive” re-ordering imports - but also finally moves local components from @/ to the bottom

only had to apply about a dozen of manual fixes

@nikgraf nikgraf requested review from Copilot and removed request for Copilot July 22, 2025 19:38
Copy link
Contributor

@Copilot 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 pull request upgrades Biome from version 1.9.4 to 2.0.6, which includes significant configuration changes and code formatting/linting rule updates. The upgrade modernizes the project's linting and formatting setup with improved rules and better organization.

  • Updates Biome configuration schema and modernizes file inclusion/exclusion patterns
  • Applies automatic code formatting fixes including import ordering and unused variable naming
  • Enhances linting rules with stricter style guidelines and accessibility improvements

Reviewed Changes

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

File Description
package.json Updates Biome dependency from 1.9.4 to 2.0.6
biome.jsonc Modernizes configuration with new schema, file inclusion patterns, and enhanced linting rules
Multiple .ts/.tsx files Applies import reordering, unused variable prefixing, and code style improvements
Multiple package.json files Formats arrays to multiline format for better readability
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Copy link
Collaborator

@cmwhited cmwhited left a comment

Choose a reason for hiding this comment

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

Didn't go through all files. But config changes look good. Thanks for doing this 🔥

@nikgraf nikgraf merged commit 411b6e8 into main Jul 22, 2025
6 checks passed
@nikgraf nikgraf deleted the ng/upgrade-biome branch July 22, 2025 19: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