Use declare let in globals.ts

[KamilaBorowska]

Should fix LGTM warnings.

[Zarel]

This pull request fixes 7 alerts when merging 6d3e724 into 5d7d317 - view on LGTM.com

fixed alerts:

• 7 for Unused variable, import, function or class

---

Comment posted by LGTM.com

[Zarel]

Neat! Unfortunately, we seem to have a whole new world of TypeScript errors due to the change.

[Zarel]

Apparently globals.ts previously just wasn't doing anything?

[sirDonovan]

Now import() is getting the types of the files while before require() just resolved to any.

[DubbleClick]

Implemented in #5139.
Can't use declare because the variable will not be assigned anywhere else, can't use import because it returns an asnyc Promise. Would have to do a whole bunch of changes to the code and wrap usages in asnyc/await blocks, so I refrained from that and just took type hints from the import while using standard require.

[Zarel]

Implemented in ffdcff3

[Zarel]

(Thanks, @DubbleClick for reminding me to do this, and also for the approach I ended up using.)