common-workflow-language / schema_salad

Semantic Annotations for Linked Avro Data

Home Page:https://www.commonwl.org/v1.2/SchemaSalad.html

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[TypeScript Codegen] Remove undefined type from optional types

ZimmerA opened this issue · comments

Currently, types marked as optional (using the ?: notation) still contain the undefined type. This does not cause any issues but is also redundant.

To fix this, the type_loader function has to be modified in a way that the undefined type is not added to the instance_type field while still retaining information about the type being optional so that functions like declare_field can add the ?: notation.