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

[chore] 테스트 CI 초기 세팅 #236

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

[chore] 테스트 CI 초기 세팅 #236

wants to merge 2 commits into from

Conversation

BellYun
Copy link
Contributor

@BellYun BellYun commented Jan 1, 2025

변경 사항

테스트 CI 초기 세팅 설정

테스트 결과 (테스트를 했으면)

테스트에 대한 CI 초기 세팅을 완료
.github/workflows/front-ci.yml에서 test 관련 ci 설정시 작동

yarn test --coverage시 커버리지 확인 가능

+α Checklist

  • 자바 코드 컨벤션을 지키면서 프로그래밍했는가?
  • 한 메서드에 오직 한 단계의 들여쓰기(indent)만 허용했는가?
  • else 예약어를 쓰지 않았는가?
  • 모든 원시값과 문자열을 포장했는가?
  • 콜렉션에 대해 일급 콜렉션을 적용했는가?
  • 3개 이상의 인스턴스 변수를 가진 클래스를 구현하지 않았는가? (가능하면 인스턴스 변수의 수를 줄이기 위해 노력한다.)
  • getter/setter 없이 구현했는가? (단, DTO는 허용한다.)
  • 메소드의 인자 수를 제한했는가? (4개 이상의 인자는 허용하지 않는다. 3개도 가능하면 줄이기 위해 노력해 본다.)
  • 코드 한 줄에 점(.)을 하나만 허용했는가?
  • 메소드가 한가지 일만 담당하도록 구현했는가?
  • 클래스를 작게 유지하기 위해 노력했는가?

@BellYun BellYun linked an issue Jan 1, 2025 that may be closed by this pull request
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review by ChatGPT

@BellYun BellYun self-assigned this Jan 1, 2025
@BellYun BellYun added 🧭 Chore 개발 환경 세팅 📁 Test 테스트 관련 코드 Frontend labels Jan 1, 2025
jung2941
jung2941 previously approved these changes Jan 3, 2025
Copy link
Collaborator

@jung2941 jung2941 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧭 Chore 개발 환경 세팅 Frontend 📁 Test 테스트 관련 코드
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[chore] 테스트 CI 적용
2 participants