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 pre-commit test to test.yaml #21

Merged
merged 3 commits into from
Nov 9, 2023
Merged

Conversation

minkyu97
Copy link
Collaborator

@minkyu97 minkyu97 commented Nov 5, 2023

PR 대상 브랜치와 비교하여 달라진 파일들에 대해 pre-commit 테스트를 github action 에서 수행하도록 변경합니다.
실수로든 아니면 환경 문제든 pre-commit 설치 하지 않거나 못하는 경우를 위해 작성했습니다.

@minkyu97 minkyu97 requested review from shp7724 and fivessun5 November 5, 2023 16:46
@minkyu97
Copy link
Collaborator Author

minkyu97 commented Nov 5, 2023

@shp7724 이거 테스트 왜 안 돌아가는지 앎?

@shp7724
Copy link
Collaborator

shp7724 commented Nov 9, 2023

진짜 왜 안돌아가지?
conflict 있어서 merge commit 생성을 못해서 그런건가

@minkyu97
Copy link
Collaborator Author

minkyu97 commented Nov 9, 2023

@shp7724 눈썰미 폼 미쳤다

@minkyu97
Copy link
Collaborator Author

minkyu97 commented Nov 9, 2023

에구 실수로 마이그레이션 테스트도 날렸네 다시 푸시함

@minkyu97 minkyu97 force-pushed the feat/github-test/formatter branch from f1df73f to bc0a193 Compare November 9, 2023 12:26
@minkyu97 minkyu97 merged commit 65a53f1 into develop Nov 9, 2023
2 checks passed
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