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.

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

Adopt struct validator more #718

Merged

Conversation

jihoon-seo
Copy link
Member

  • 나머지 MCIR (SecurityGroup, Spec, SSHKey, vNet) 에 대해서도 validator를 도입했습니다.
    • 잘 적용되었는지 (struct tag, validation 함수, 빠진 필드가 없는지 등) 검토가 이루어지면 좋습니다.
  • MCIS의 *Req struct 중, TbMcisReq, TbVmReq, McisCmdReq, MonAgentInstallReq 에 대해서도 validator를 도입했습니다.
    • 잘 적용되었는지 (struct tag, validation 함수, 빠진 필드가 없는지 등) 검토가 이루어지면 좋습니다.
    • MCIS의 *Req struct 중, 나머지 struct에도 적용이 필요한지 검토가 이루어지면 좋습니다.

@seokho-son
Copy link
Member

ref #706

@seokho-son
Copy link
Member

@jihoon-seo 이대로 좋은 듯 합니다! ^^ 감사감사.

MCIS의 *Req struct 중, 나머지 struct에도 적용이 필요한지 검토가 이루어지면 좋습니다.

도 당연히 적용되면 좋을 것 같네요. (required 여부 지정은 잘 모르겠지만, 틀을 제공하는 측면에서)

PR은 일단 머지하도록 할까요? ^^

@seokho-son
Copy link
Member

/approve

@seokho-son seokho-son merged commit bc8e114 into cloud-barista:main Sep 9, 2021
@jihoon-seo jihoon-seo deleted the 210908_Adopt_struct_validator_more branch September 9, 2021 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants