Improve TS types to support isolatedModules and isolatedDeclarations#9029
Improve TS types to support isolatedModules and isolatedDeclarations#9029bjornharrtell wants to merge 4 commits intomasterfrom
Conversation
|
The large amount of changes are due to that the the files has been re-generated with the code generator. It should have amounted to much smaller difference, but I see generated code has been subjected to automated formatting in #8707. Did that intentionally also apply for generated code @dbaileychess? I find it hard to understand rationale for that, or at least it would mean it should be applied again after new code gen but I'm not sure the tooling for it is available outside google. |
|
I'm not sure how to go about reviewing this given the large scale of changes. Do you have a quick tl;dr on the non code gen changes? |
|
@jtdavis777 issue reference in the PR description was wrong. Fixed now (#9028) and with rationale for the changes. As for the relevant manual changes here it's in the ts code gen (src/idl_gen_ts.cpp) adding explicit declaration and return types. Additionally for the non generated typescript implementation same kind of fixes has been done manually (in ts/flexbuffers/stack-value.ts, ts/flatbuffers.ts and ts/utils.ts). Remaining diff is due to code gen which looks extreme due to #8707. |
ref #9028