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

최소 테스트 커버리지 증가 #59

Merged
merged 1 commit into from
Nov 8, 2024
Merged

최소 테스트 커버리지 증가 #59

merged 1 commit into from
Nov 8, 2024

Conversation

DaleSeo
Copy link
Contributor

@DaleSeo DaleSeo commented Nov 7, 2024

프로젝트 막 셋업햇을 때는 테스트가 별로 없어서 최소 70% 커버리지의 요구사항을 맞추도록 Vitest 설정을 할 수가 없었습니다. 참고: #22 (comment)

하지만 그 동안 팀원 분들이 충실히 테스트를 작성해주셔서 현재는 넉넉하게 80%를 넘기고 있는 상황입니다. 그래서 앞으로 계속 이 상태를 유지하기 위해서 Vitest의 coverage threshhold를 목표치인 70%로 늘리도록 하겠습니다.

Shot 2024-11-07 at 15 50 36@2x

@DaleSeo DaleSeo requested a review from a team as a code owner November 7, 2024 20:54
@DaleSeo DaleSeo merged commit 035ac04 into main Nov 8, 2024
2 checks passed
@DaleSeo DaleSeo deleted the bump-up-coverage branch November 8, 2024 12:46
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