forked from shirsig/WIM
-
Notifications
You must be signed in to change notification settings - Fork 3
Closed as not planned
Labels
enhancementNew feature or requestNew feature or request
Description
Summary
Add LuaCheck to catch bugs and code quality issues via static analysis.
Motivation
LuaCheck can detect real bugs such as accidental globals, unused variables, and variable shadowing. Some of the recently fixed issues (closure bugs in hook loops, unsafe table modification during iteration) are the kind of problems that static analysis can catch early.
Plan
- Add a
.luacheckrcwith:- Lua 5.1 standard (WoW Lua version)
- WoW API globals allowlist (there are community-maintained lists for this, e.g. from wow-declarations or wowless)
- WIM-specific globals
- Fix or suppress any initial findings
- Document how to run it locally
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request