Always require data for `EventEmitter.fire`
Always require data for `EventEmitter.fire`

Fixes #93875

The data field should not be optional as this breaks strict null typings. With `EventEmitter<void>`, you can still call `.fire()` with no arguments

Microsoft/vscode,Microsoft/vscode,microsoft/vscode,Krzysztof-Cieslak/vscode,Microsoft/vscode,hoovercj/vscode,hoovercj/vscode,eamodio/vscode,eamodio/vscode,microsoft/vscode,Microsoft/vscode,the-ress/vscode,microsoft/vscode,Microsoft/vscode,hoovercj/vscode,the-ress/vscode,hoovercj/vscode,microsoft/vscode,Krzysztof-Cieslak/vscode,eamodio/vscode,microsoft/vscode,Krzysztof-Cieslak/vscode,the-ress/vscode,microsoft/vscode,Microsoft/vscode,Microsoft/vscode,microsoft/vscode,Krzysztof-Cieslak/vscode,Microsoft/vscode,hoovercj/vscode,eamodio/vscode,the-ress/vscode,Krzysztof-Cieslak/vscode,Microsoft/vscode,hoovercj/vscode,Krzysztof-Cieslak/vscode,the-ress/vscode,Microsoft/vscode,microsoft/vscode,Microsoft/vscode,the-ress/vscode,microsoft/vscode,hoovercj/vscode,eamodio/vscode,microsoft/vscode,Krzysztof-Cieslak/vscode,Krzysztof-Cieslak/vscode,hoovercj/vscode,Krzysztof-Cieslak/vscode,microsoft/vscode,eamodio/vscode,Krzysztof-Cieslak/vscode,the-ress/vscode,Microsoft/vscode,eamodio/vscode,the-ress/vscode,Microsoft/vscode,the-ress/vscode,eamodio/vscode,eamodio/vscode,hoovercj/vscode,eamodio/vscode,eamodio/vscode,the-ress/vscode,the-ress/vscode,the-ress/vscode,microsoft/vscode,hoovercj/vscode,hoovercj/vscode,eamodio/vscode,microsoft/vscode,Krzysztof-Cieslak/vscode,Krzysztof-Cieslak/vscode,microsoft/vscode,Microsoft/vscode,hoovercj/vscode,Krzysztof-Cieslak/vscode,microsoft/vscode,Microsoft/vscode,eamodio/vscode,Krzysztof-Cieslak/vscode,Krzysztof-Cieslak/vscode,the-ress/vscode,hoovercj/vscode,Krzysztof-Cieslak/vscode,hoovercj/vscode,eamodio/vscode,the-ress/vscode,Microsoft/vscode,hoovercj/vscode,the-ress/vscode,Microsoft/vscode,microsoft/vscode,the-ress/vscode,hoovercj/vscode,eamodio/vscode,Krzysztof-Cieslak/vscode,hoovercj/vscode,eamodio/vscode,eamodio/vscode,microsoft/vscode