Resolve #1027 - [Alibaba] When creating VM Spec, I always got an unnecessary error message. #1080
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[Alibaba] When creating VM Spec, I always got an unnecessary error message. 제조사 정보 추출: 원문[%s] / 추출[%s] -> Manufacturer Information Extraction: Original[%s] / Extracted[%s] 조회된 인스턴스 타입 수 -> Number of Retrieved Instance Types 스펙 정보 조회 -> Retrieve Specification Information 에 해당하는 Spec 정보를 찾을 수 없습니다. -> Cannot find Spec information for the specified item. cblogger.Infof 한글 로그 출력 결과 영문으로 변경 완료. 불필요 로그 메세지 제거 완료.
불필요 에러 메시지 제거 완료 한글 -> 영문으로 변경 완료
|
yunkon-kim
reviewed
Feb 19, 2024
| @@ -81,7 +81,7 @@ func ExtractVMSpecInfo(Region string, instanceTypeInfo *ec2.InstanceTypeInfo) ir | |||
| //KeyValue 목록 처리 | |||
| keyValueList, errKeyValue := ConvertKeyValueList(instanceTypeInfo) | |||
| if errKeyValue != nil { | |||
| cblogger.Errorf("[%]의 KeyValue 추출 실패", *instanceTypeInfo.InstanceType) | |||
| // cblogger.Errorf("[%]의 KeyValue 추출 실패", *instanceTypeInfo.InstanceType) | |||
There was a problem hiding this comment.
@thdclals18 안녕하세요.
Korean-checker 워크플로우는 CB-Spider의 한글 사용정책과 관련하여 도움을 드리는 도구입니다.
해당 라인과 같이 변경된 부분에서 의도하지 않게 한글이 쓰인 경우를 탐지해서 기여/리뷰에 도움을 드리고 있습니다.
우선 이번 PR에서는 이 부분의 한글을 수정해 주시기 바랍니다 ^^
(참고)
본래는 워크플로우에서 위와 같은 내용을 코멘트로 남겨주어야 하는데요. 권한 및 브랜치와 관련한 이슈로, 워크플로우에 에러가 발생하였습니다. 😓 (이 부분은 신경 쓰지 않으시길 바랍니다.)
워크플로우는 쌍따옴표("") 사이에 한글을 검사하는 로직이 들어가 있습니다. 주석에 ""이 포함되어있는 경우 검사 대상이 됩니다. 이 부분은 정책상 선택적으로 수정하시면 될 것 같습니다.
워크플로우가 정상적으로 동작하는 시점부터는 // "한글"과 같은 경우에도 메인테이너께서 판단 후 머지하실 수 있습니다. 참고하시기 바랍니다. (CC @powerkimhub)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
**이슈 (#1027)
작업내용