Hold off for a bit, I just encountered another error when actually trying to deserialize this endpoint toWorkflowRunsList.

It's odd, in the response bothlicenses andrepo_transfer are being returned as null, but they arerequired in the OpenAPI spec:

"has_wiki":true,"has_pull_requests":true,"has_projects":true,"projects_mode":"all","has_releases":true,"has_packages":true,"has_actions":true,"ignore_whitespace_conflicts":false,"allow_merge_commits":true,"allow_rebase":true,"allow_rebase_explicit":true,"allow_squash_merge":true,"allow_fast_forward_only_merge":true,"allow_rebase_update":true,"allow_manual_merge":false,"autodetect_manual_merge":false,"default_delete_branch_after_merge":false,"default_merge_style":"merge","default_allow_maintainer_edit":false,"avatar_url":"","internal":false,"mirror_interval":"","object_format_name":"sha1","mirror_updated":"0001-01-01T00:00:00Z","repo_transfer":null,"topics": [],"licenses":null

Fixed the model to omit null (but not nullable) values and initialize the Licenses array in the response even if they are empty.

Forgot that the original didn't have the pre-allocation. Your suggestion should be ideal.

Forgot that the original didn't have the pre-allocation. Your suggestion should be ideal.

For performance-critical code, thatif and pre-allocation does help.

While here I think we can keepStringList as simple as before. TBH I think in this case there should be no noticeable difference, while simple code reads better.

(I just meant that in the convert function, we can't depend on the StringList's behavior)