Closed graup closed 5 years ago
This is a fix for #98. It builds on top of #92 (should be merged first). This works by recursively calling make_type_name when we have typeArguments.
make_type_name
typeArguments
This PR additionally changes the order of fields (they should come before methods) by using the longname for sorting instead of just the name.
longname
I also added a build test for typescript.
Let's merge #92 first and then review this PR separately.
I rebased this on the current master and cleaned up the commit history.
Summary of changes:
@erikrose Here's a gentle reminder to look at this PR when you've got some time. Thanks.
This is a fix for #98. It builds on top of #92 (should be merged first). This works by recursively calling
make_type_name
when we havetypeArguments
.This PR additionally changes the order of fields (they should come before methods) by using the
longname
for sorting instead of just the name.I also added a build test for typescript.
Let's merge #92 first and then review this PR separately.