Skip to content

feat(deathreport): convert package to --!strict#719

Merged
Quenty merged 3 commits into
mainfrom
users/james/deathreport-strict-typing
Jun 24, 2026
Merged

feat(deathreport): convert package to --!strict#719
Quenty merged 3 commits into
mainfrom
users/james/deathreport-strict-typing

Conversation

@Quenty

@Quenty Quenty commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Converts every non-strict source file in the deathreport package to --!strict, adding explicit type annotations across the death, kill, and team tracker classes, DeathReportUtils, and the client service. Binder registrations hoist their class requires to locals so the sanctioned any cast doesn't trip luau-lsp's require resolver. There are no runtime behavior changes.

Converts every non-strict source file in the deathreport package to --!strict,
adding explicit type annotations to the death/kill/team tracker classes,
DeathReportUtils, and the client service. Binder registrations now hoist their
class requires to locals so the sanctioned `:: any` cast doesn't trip luau-lsp's
require resolver. No runtime behavior changes.
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

Test Results

Package Status Try it
@quenty/deathreport ✅ Passed (4/4) (751ms) Open | Play

1 tested, 1 passed, 0 failed in 5.7s · View logs

Deploy Results

ℹ️ No changed packages with deploy targets were discovered for this PR. · View logs

@Quenty Quenty merged commit 238945a into main Jun 24, 2026
9 checks passed
@Quenty Quenty deleted the users/james/deathreport-strict-typing branch June 24, 2026 19:34
@Quenty

Quenty commented Jun 24, 2026

Copy link
Copy Markdown
Owner Author

🚀 PR was released in @quenty/deathreport@10.45.0 🚀

@Quenty Quenty added the released This issue/pull request has been released. label Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released This issue/pull request has been released.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant