Support strictNullChecks

[miniak]

Example

class BrowserWindow extends EventEmitter {
    ...
    setMenu(menu: Menu): void;
    ...
}

should be

class BrowserWindow extends EventEmitter {
    ...
    setMenu(menu: Menu | null): void;
    ...
}

The whole API needs to be reviewed and instances where an instance can be null need to be marked.

[MarshallOfSound]

If the Electron docs are updated with the | syntax the linter supports then this will Just Work(tm).

E.g. Menu | null

[mike-morr]

Definitely need this!

[sindresorhus]

It should use optional parameters instead:

class BrowserWindow extends EventEmitter {
    ...
    setMenu(menu?: Menu): void;
    ...
}

[MarshallOfSound]

@sindresorhus I believe in most cases null is explicitly required for these methods to indicate a falsey value.

Better APIs could be written E.g. delete window.menu or window.menu = null or window.removeMenu() but that isn't a TS issue. The current types of Menu | null is correct there.

Also I'm like 99% sure we can close this as these menu types (and other types) have been updated to be | null where appropriate. (Or at least can be so the generator is 👍 here)