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

2주차 - 김정호 #12

Merged
merged 26 commits into from
Jan 9, 2023
Merged

2주차 - 김정호 #12

merged 26 commits into from
Jan 9, 2023

Conversation

hou27
Copy link
Member

@hou27 hou27 commented Jan 8, 2023

No description provided.

application.properties -> application.yml로 파일명 변경
- 메서드 이름은?
- 덧셈 기능을 제공하는 메서드의 파라미터 개수는?
- 메서드는 정적 메서드? 인스턴스 메서드?
- 메서드를 제공하는 클래스명은?
모든 규칙을 충족하는 경우
PasswordStrengthMeter 작성
PasswordStrength 작성
아무 조건도 충족하지 못한 경우를 위해서
조건문 수정 및 반환문 순서 조정
가독성 개선을 위해 충족한 조건 개수 세는 코드 메서드로 분리
@hou27 hou27 added this to the GDSC MJU 1기 TDD 스터디 milestone Jan 8, 2023
@hou27 hou27 requested a review from rladuswl January 8, 2023 14:59
@hou27 hou27 self-assigned this Jan 8, 2023
@hou27 hou27 linked an issue Jan 8, 2023 that may be closed by this pull request
3 tasks
@hou27 hou27 changed the title 1주차 - 김정호 2주차 - 김정호 Jan 9, 2023
Copy link
Collaborator

@rladuswl rladuswl left a comment

Choose a reason for hiding this comment

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

머지하겠습니다~

@rladuswl rladuswl merged commit e13bc0a into main Jan 9, 2023
@hou27 hou27 linked an issue Jan 10, 2023 that may be closed by this pull request
4 tasks
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주차 - 테스트 코드 작성 순서 - 김정호 2주차 - TDD 시작 - 김정호
2 participants