Skip to content

Conversation

@DayKev
Copy link
Collaborator

@DayKev DayKev commented Dec 4, 2025

What are the changes the user will see?

N/A

Why am I making these changes?

There shouldn't be 2 different directories for types.

What are the changes from a developer perspective?

  • Moved files from src/typings/ to src/@types/ and renamed phaser/index.d.ts to phaser.d.ts.
  • Consolidated API types files into api.ts.
  • Consolidated save migrator types into @types/save-migrators.ts.
  • Deprecated daily scoreboard stuff.

Checklist

  • 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 requested a review from a team as a code owner December 4, 2025 11:07
@DayKev DayKev added the Refactor Rewriting existing code related label Dec 4, 2025
@DayKev DayKev force-pushed the refactor/consolidate-types branch 2 times, most recently from 085e23d to c950e2b Compare December 4, 2025 11:15
@DayKev DayKev enabled auto-merge (squash) December 5, 2025 00:17
@DayKev DayKev force-pushed the refactor/consolidate-types branch from c950e2b to 8be0699 Compare December 5, 2025 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactor Rewriting existing code related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants