Make VNode generic type param optional

I suggest we make the VNode generic parameter optional by providing a default type for it.

Adding a generic parameter to the VNode interface breaks the types of many preact libraries that reference or extend VNode. Until many of these libraries update their types to provide a value for the VNode type parameter, I suggest we make it optional. Making it optional unblocks Preact TypeScript users to adopt 8.2.8+ without having to wait for all other libraries to catch up.

Below are some of the libraries that I’ve found that would need updating. I’ll update this list with PRs to take advantage of the type parameter once they are opened. Post a comment here if you open one.

Libraries that need updating to provide a type param to VNode:

• developit/preact-router
• developit/preact-render-to-string
• mzgoddard/preact-render-spy

[marvinhagemeister]

This is fixed in master. The generic parameter vor VNode is marked as optional. That way we don’t need to update all other libraries depending on preact 🎉

[andrewiggins]

FYI, I’m working through fixing issues that arise while trying to convert the preact-cli default template to TypeScript. The goal is to enable the default template to be written in TypeScript strict mode without any errors. My repo is at andrewiggins/preact-default-ts