Skip to content

[Protobuf-Schema] Add unknown to enum switch#10892

Merged
wing328 merged 8 commits into
OpenAPITools:masterfrom
Yurzel:protobuf-add-unknown-enum
Nov 22, 2021
Merged

[Protobuf-Schema] Add unknown to enum switch#10892
wing328 merged 8 commits into
OpenAPITools:masterfrom
Yurzel:protobuf-add-unknown-enum

Conversation

@Yurzel

@Yurzel Yurzel commented Nov 18, 2021

Copy link
Copy Markdown
Contributor

resolves #10891

Codegen

  • New "startEnumsWithUnknown" switch (default is false)

@wing328

wing328 commented Nov 20, 2021

Copy link
Copy Markdown
Member

Thanks for the PR but your commit (as shown in the Commits tab) is not linked to your Github account, which means this PR won't count as your contribution in https://github.com/OpenAPITools/openapi-generator/graphs/contributors.

Let me know if you need help fixing it.

Ref: https://github.com/OpenAPITools/openapi-generator/wiki/FAQ#how-can-i-update-commits-that-are-not-linked-to-my-github-account

@wing328

wing328 commented Nov 22, 2021

Copy link
Copy Markdown
Member

I just merge another PR submitted by you. Can you please resolve the merge conflicts in this PR when you've time?

@Yurzel

Yurzel commented Nov 22, 2021

Copy link
Copy Markdown
Contributor Author

Done.

I forgot to generate docs, so thats done aswell. The last PR will have conflicts aswell, so I'll fix that one too when you merge this.

@wing328 wing328 merged commit c13067d into OpenAPITools:master Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REQ][Protobuf-Schema] Add unknown to enum (index from 1)

2 participants