Skip to content

Remove unused local bindings - #24

Open
ansien wants to merge 1 commit into
MadStudioRoblox:mainfrom
ansien:codex/remove-unused-locals
Open

Remove unused local bindings#24
ansien wants to merge 1 commit into
MadStudioRoblox:mainfrom
ansien:codex/remove-unused-locals

Conversation

@ansien

@ansien ansien commented Sep 3, 2026

Copy link
Copy Markdown

Removes three Luau lint diagnostics by deleting the unused success = true declaration immediately shadowed by the following pcall result, and dropping the unused error_message binding from the page-advance pcall.

Both pcall calls, their consumed success values, error handling, and queue cleanup remain unchanged.

Validation:

  • git diff --check passes.
  • Luau LSP 1.69.0 with Roblox definitions reports two LocalUnused warnings and one LocalShadow warning before this patch, and none of those warnings afterward.
  • The same consumer analysis setup reports 98 type errors both before and after this patch; this change does not claim to resolve those or make full analysis pass.
  • No Studio or DataStore execution was performed for this lint-only change.

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.

1 participant