Skip to content

Comments

dev: update Biome from 2.3.4 to 2.3.8 and add some rules#1392

Merged
DayKev merged 15 commits intobetafrom
dev/update-biome-2.3.7
Dec 11, 2025
Merged

dev: update Biome from 2.3.4 to 2.3.8 and add some rules#1392
DayKev merged 15 commits intobetafrom
dev/update-biome-2.3.7

Conversation

@DayKev
Copy link
Contributor

@DayKev DayKev commented Nov 26, 2025

What are the changes the user will see?

N/A

Why am I making these changes?

Keep deps up-to-date, improve codebase with lint rules.
After Biome is updated, most of the noUnusedImports suppressions will be removed (except for 5, due to a bug where TSDoc usage on object properties is not detected).

What are the changes from a developer perspective?

  • The lint/style/noNestedTernary rule was fixed in 2.3.6 to properly detect nested ternaries within parentheses, requiring updating the codebase to fix the newly-detected issues.
  • Some TSDocs and parameter names were updated in encounter-pokemon-utils.ts while fixing the nearby lint issues.
  • Parsing of test functions for linting was fixed in 2.3.6 which incidentally caused the formatter to format them differently. Changes were minimized via specifically placed comments and suppression comments.

New Rules

Nursery

Correctness

Checklist

  • Otherwise: I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I tested the changes manually?

@DayKev DayKev self-assigned this Nov 26, 2025
@DayKev DayKev added Dependencies Pull requests that update a dependency file Development Changes designed to improve the development process labels Nov 26, 2025
@DayKev DayKev marked this pull request as draft November 26, 2025 02:13
@DayKev

This comment was marked as resolved.

@DayKev DayKev force-pushed the dev/update-biome-2.3.7 branch from 540b93b to 33f4b0e Compare November 26, 2025 08:07
@DayKev DayKev marked this pull request as ready for review November 26, 2025 08:07
@DayKev DayKev changed the title dev: update Biome from 2.3.4 to 2.3.7 and add some rules dev: update Biome from 2.3.4 to 2.3.8 and add some rules Nov 28, 2025
@DayKev DayKev force-pushed the dev/update-biome-2.3.7 branch 4 times, most recently from 4a80233 to a066717 Compare December 9, 2025 14:00
@DayKev DayKev force-pushed the dev/update-biome-2.3.7 branch from a066717 to 95d5840 Compare December 10, 2025 13:41
Tempo-anon
Tempo-anon previously approved these changes Dec 11, 2025
Remove now-obsolete `noUnusedImport` suppressions

There's a bug where TSDoc usage on object properties isn't detected,
leaving 4 files with suppressions still until the bug is fixed:
`battle-scene-event-type.ts`, `challenge-type.ts`,
`egg-event-type.ts`, and `move-flags.ts`; all in the `enums/` directory
@DayKev DayKev merged commit 8866c0d into beta Dec 11, 2025
9 checks passed
@DayKev DayKev deleted the dev/update-biome-2.3.7 branch December 11, 2025 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dependencies Pull requests that update a dependency file Development Changes designed to improve the development process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants