Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

This way can not parse the first value of enum for some reasons #3628

Closed
pipecd-bot opened this issue May 13, 2022 · 0 comments
Closed

This way can not parse the first value of enum for some reasons #3628

pipecd-bot opened this issue May 13, 2022 · 0 comments
Assignees
@knanao
Labels
kind/todo

Comments

@pipecd-bot
Copy link
Collaborator

pipecd/tool/codegen/protoc-gen-auth/main.go

Lines 107 to 110 in 722b591

// FIXME: This way can not parse the first value of enum for some reasons hence
// set VIEWER for default value.
method.Role = "VIEWER"
if v.String() != "" {

This was created by todo plugin since "FIXME:" was found in 722b591 when #3623 was merged. cc: @knanao.
@knanao knanao changed the title This way can not parse the first value of enum for some reasons hence This way can not parse the first value of enum for some reasons May 17, 2022
@knanao knanao closed this as completed Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@knanao knanao

Labels
kind/todo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@knanao @pipecd-bot