Move all validation out of `Scope.ts` and into the ScopeValidator

rokucommunity / brighterscript

A superset of Roku's BrightScript language

MIT License

159 stars 46 forks source link

Move all validation out of `Scope.ts` and into the ScopeValidator #1093

Closed TwitchBronBron closed 6 months ago

TwitchBronBron commented 6 months ago

We should move all the scope validation out of Scope.ts and into ScopeValidator. This might be a breaking change since we should just delete Scope.validate() in favor of the plugin system hooks.

Maybe do the same for other items (like BrsFile, XmlFile, XmlScope, etc...). Let's fully leverage the plugin event hooks and eliminate those functions