rokucommunity / brighterscript / #5038

88%

master: 89%

push

Remove function scope and improve type tracking (#401)

* Add `for each` variable to completions

* ForEachStatement's `item` is now VaribleExpression

* Fix ts build issue.

* fix broken tests

* fix broken test.

* Collection function variables in _references

* Removed FunctionScope in favor of func expressions

* lint and interface fixes

* Repopulate `references.localVars` during `findReferences`

* Handle sub with explicit return type

* Fix `findReferences` for local vars

* Normalize line endings for Scope.ts

* Feature/symbol table (#380)

* Ran npm audit and fixed vulvnerability

* Added symbol table class

* Parser adds to a symbol table as it parses code

* Revert package changes

* Fixed test that was referencing LazyType when it was not supposed to

* Better handling of a variable potentially changing type. Added getSymbolType() function and fixed error of a function inheriting symbols from a containing function

* Fixed typo

* Removed code comment

* Fixes from PR comments - and made sure for each loop variables were added to the symbolTable

* Changed BscSymbol interface to include a range

Co-authored-by: Mark Pearce <mark.pearce@redspace.com>

* Feature/scope tracked symbol tables (#395)

* Adds SymbolTable.hasKey method

* Makes CustomType display its actual type

* Standardize how a FunctionType is created from an expression

* Adds symbol tables to namespaces

* Adds a symbol table to scope, and computes merged symbol tables across files

* Use the symbolTable on getHover

* Working on argument type mismatch validation

* Proper type tracking across files in the same scope, including namespaces. Adds validation on argument types

* Fixed bad whitespace in tests

* Updates from code review

* Fixed all circular dependency issues

* Hopefully fixed all whitespace issues

* Fixes for when scope changes

* Fixed lint error and remove symbolTable clear - it wa... (continued)

4017 of 4960 branches covered (80.99%)

Branch coverage included in aggregate %.

6704 of 7389 relevant lines covered (90.73%)

1077.13 hits per line