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

Add lint test for WorkflowTemplate #33

Merged
merged 1 commit into from
Jan 21, 2022
Merged

Add lint test for WorkflowTemplate #33

merged 1 commit into from
Jan 21, 2022

Conversation

Jaesang
Copy link
Contributor

@Jaesang Jaesang commented Jan 21, 2022

tks-flow의 WorkflowTemplate의 대한 Lint Test 입니다.
Pull Request 시 변경 파일에 대해 Lint Test를 수행합니다.

resolve openinfradev/tks-issues#96
@Jaesang
Copy link
Contributor Author

Jaesang commented Jan 21, 2022

이 Action에 대한 테스트 PR입니다.
Jaesang/argo-workflow#1

테스트 PR에서 수행한 Lint Test 결과입니다.
https://github.com/Jaesang/argo-workflow/runs/4891214027?check_suite_focus=true

@robertchoi80
Copy link
Contributor

두가지 comment가 있습니다.

  • 우선, argo template lint 명령어가 어느 정도까지 lint를 해주는지 궁금한데, 테스트하신 repo에 일부러 lint rule 에 위반되는 commit 등을 하고나서 결과 보여주시면 좀더 체감이 될 것 같습니다.
  • 수행결과 보니 Argo CLI 설치하는데 1초 정도밖에 안걸려서 이정도는 크게 상관 없겠지만, 향후 좀더 heavy한 설치 과정이 필요하게 되면 그때는 이미 필요한 tool들이 설치된 custom image 사용을 고려해볼 필요도 있을것 같습니다. https://github.community/t/how-do-i-use-custom-docker-image-with-github-actions/18112 (마지막 코멘트에 공식 doc 쪽 링크가 걸려있습니다)

@Jaesang
Copy link
Contributor Author

Jaesang commented Jan 21, 2022

@robertchoi80

@zugwan zugwan merged commit 079142f into openinfradev:main Jan 21, 2022
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.

3 participants