fa icon indicating copy to clipboard operation
fa copied to clipboard

Annotation fixes

Open clyfordv opened this issue 10 months ago • 1 comments

Goal: It would be nice for the warnings given by the lua language server to be more limited and useful, instead of everywhere, all the time.

This is a test run for eliminating warnings through annotation fixes. In some cases functions have been moved around, or nil checks have been added. Logic has otherwise not been modified.

For the largest commit (for eliminating warnings in the BaseManager files), the other modified files contained functions or classes where annotation fixes have eliminated warnings.

The warnings that are left are (in theory) the real ones--deprecated functions, junky code, etc. I want to make sure I'm on the right track before chiseling away at this more.

clyfordv avatar Apr 04 '24 06:04 clyfordv

@clyfordv we specifically moved away from Integer to number hence why that their @Garanas can you remember why we did this?

MrRowey avatar Jun 19 '24 09:06 MrRowey