fix type exports in typescript 4.5

erikbrinkman commented 2 years ago

Typescript 4.5 adds type modifiers for import / export lists.

This PR addresses that for export lists by keeping most of the code the same, but using the type keyword to split value lists. Since I don't think flow has a corresponding syntax, it's currently split into two export declarations if such a list exists.

Caveats:

I'm not an expert in flow or this library so harsh critique is welcome
This syntax applies to import as well. I only need support for export, so that's all I've added, but if you really want I can look into full support
I saw a todo to abstract printing into the printers module. I don't really want to do that in this PR, but a plus is in the current form, this PR doesn't seem to make that any harder

erikbrinkman commented 2 years ago

@joarwilk gentle ping on this, I don't want to have to separate all my exports to get this to work

orta commented 2 years ago

I missed this PR, yeah, this conceptually looks good to me 👍🏻

orta commented 2 years ago

Shipped in 1.18

joarwilk / flowgen

fix type exports in typescript 4.5 #175