Closed jihoon-seo closed 3 years ago
PR 을 올리고 나서 보니, CamelCase 에서 대소문자 일관성이 떨어지는 부분이 있습니다.
=> 수정했습니다.
CamelCase 수정하는 commit 을 작성하고 (Fix lint errors (2nd commit)
)
이전 commit (Fix lint errors
) 에 squash 하고
force-push 했습니다.
@jihoon-seo
빠르게 적용하고 계시네요 ㅎㅎ
감사하면서도 아직 미흡한 부분으로 인한 에러로 죄송하기도 합니다. ㅠㅠ (현재로서는 cloud-barista/cb-operator에 바로 적용하시기 보다 Fork하여 테스트 해보시는것을 추천 드립니다.)
말씀하신 에러에 대한 파일명과 라인번호 표기는 꼭 필요한 사항입니다.
저도 테스트시 Staticcheck's Checks를 참고하여 "시도 2"와 같은 방법으로 해결한 후 진행하지 못하고 있던 부분 입니다.
이슈로 등록하였고요. 업데이트 하도록 하겠습니다.
참고로, golangci-lint-action의 이슈 #119에서 ::
Syntax를 활용하여 파일명과 라인번호를 기재하였고, 이로 이 github actions에서 정보를 보여주고 있지 않다고 합니다.
@cloud-barista/cb-operator-maintainer 팀을 만들고 cb-operator 저장소 maintainer 권한을 추가하였습니다. (https://github.com/orgs/cloud-barista/teams/cb-operator-maintainer , @hermitkim1 환영 합니다..^^)
@hermitkim1 알려주신 대로 수정했습니다. 감사합니다!
LGTM
(이 PR의 Reviewers 로 @hermitkim1 님도 추가하려고 했는데, 추가가 안 되네요.. :( )
GitHub workflows 'lint' and 'build' YAML files by @hermitkim1
GitHub Lint action 실행 시 내부적으로
golangci-lint
가 실행되는데,--out-format=github-actions
옵션이 붙어서 실행되어, GitHub 웹 인터페이스에서는 어느 파일의 어느 라인이 문제인지 알 수가 없습니다. [링크]golint
를 설치하고 실행하여 표시되는 사항들을 모두 수정했으나, GitHub Lint action 의golangci-lint
는 여전히 에러가 1개 있다고 표시했습니다.golangci-lint
를 설치하고 실행하여 표시되는 사항들을 모두 수정했더니, GitHub Lint action 도 통과했습니다.--out-format=github-actions
옵션을 뺄 수 있을까 했는데,lint-on-push.yml
파일에는 해당 구문이 없습니다.