refactor: 엔티티 필드 및 생성자 메서드 시그니처 변경 #850
Conversation
워크스루이 풀 리퀘스트는 여러 도메인 엔티티의 필드 순서와 메서드 시그니처를 재구성하는 리팩토링 작업입니다. 주요 변경 사항은 엔티티 클래스들의 필드 선언 순서, 생성자 파라미터 순서, 그리고 일부 필드 추가 및 제거를 포함합니다. 변경의 목적은 코드의 일관성을 높이고 엔티티 구조를 표준화하는 것입니다. 변경 사항
이슈 평가
관련 가능성 있는 PR
추천 리뷰어
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (11)
🚧 Files skipped from review as they are similar to previous changes (8)
🔇 Additional comments (9)
✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (7)
src/main/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRound.java (1)
49-60: 생성자 매개변수 순서가 개선되었습니다.필드 선언 순서와 동일하게 매개변수가 정렬되어 일관성이 향상되었습니다. 다만, 가독성을 위해 매개변수 그룹 사이에 빈 줄을 추가하는 것을 고려해보세요.
private RecruitmentRound( String name, RoundType roundType, final Period period, Recruitment recruitment, + Integer academicYear, SemesterType semesterType) {src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyAchievement.java (1)
51-53: 팩토리 메서드가 적절히 수정되었습니다!생성자와 동일한 순서로 파라미터가 변경되어 일관성이 유지되었습니다.
메서드의 의도를 더 명확히 하기 위해 다음과 같이 JavaDoc을 추가하는 것을 고려해보세요:
+ /** + * StudyAchievement 엔티티를 생성합니다. + * + * @param achievementType 학습 성취 유형 + * @param student 학생 멤버 + * @param study 스터디 + * @return 생성된 StudyAchievement 엔티티 + */ public static StudyAchievement create(AchievementType achievementType, Member student, Study study) {src/main/java/com/gdschongik/gdsc/domain/recruitment/domain/Recruitment.java (2)
23-24: 필드 유효성 검사 추가 필요
feeName필드에 JPA 유효성 검사를 추가하는 것이 좋겠습니다. 데이터 무결성을 보장하기 위해 다음과 같은 제약 조건을 고려해 보세요:- private String feeName; + @Column(name = "fee_name", length = 100, nullable = false) + private String feeName;
41-48: 팩토리 메서드가 잘 구현되었습니다.매개변수 순서가 생성자와 일관성 있게 변경되었습니다. 다만, 메서드의 용도를 명확히 하기 위해 JavaDoc 추가를 제안드립니다.
+ /** + * Recruitment 엔티티를 생성합니다. + * @param feeName 납부금 이름 + * @param fee 납부금액 + * @param semesterPeriod 학기 기간 + * @param academicYear 학년도 + * @param semesterType 학기 구분 + * @return 생성된 Recruitment 엔티티 + */ public static Recruitment create(src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java (1)
67-83: 생성자 매개변수 순서도 필드 순서와 일치시켜야 합니다.필드 순서를 재정렬할 때 생성자의 매개변수 순서도 동일하게 맞춰주시면 코드의 일관성이 향상될 것 같습니다.
private Order( String nanoId, + String paymentKey, + ZonedDateTime approvedAt, + ZonedDateTime canceledAt, + OrderStatus status, + MoneyInfo moneyInfo, Long memberId, Long membershipId, Long recruitmentRoundId, - Long issuedCouponId, - OrderStatus status, - MoneyInfo moneyInfo) { + Long issuedCouponId) {src/main/java/com/gdschongik/gdsc/domain/study/domain/AssignmentHistory.java (1)
64-71: 생성자 매개변수 순서를 필드 순서와 일치시키는 것이 좋겠습니다.현재 필드는 다음 순서로 잘 정리되어 있습니다:
- ID
- 일반 필드 (submissionLink, commitHash, contentLength, committedAt)
- Enum 필드 (submissionStatus, submissionFailureType)
- 관계 필드 (studyDetail, member)
하지만 생성자의 매개변수 순서는 이와 다릅니다. 일관성을 위해 다음과 같이 변경하는 것을 제안합니다:
private AssignmentHistory( - AssignmentSubmissionStatus submissionStatus, - SubmissionFailureType submissionFailureType, - StudyDetail studyDetail, - Member member) { + StudyDetail studyDetail, + Member member, + AssignmentSubmissionStatus submissionStatus, + SubmissionFailureType submissionFailureType) { this.submissionStatus = submissionStatus; this.submissionFailureType = submissionFailureType; this.studyDetail = studyDetail; this.member = member; }src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyDetail.java (1)
54-56: 엔티티 필드 순서 개선 필요PR 목적에 따라 관계 필드는 마지막에 위치해야 하지만, 현재는 다른 필드들 사이에 위치해 있습니다. 필드 순서를 다음과 같이 정리하는 것이 좋습니다:
- Id 필드
- 일반 필드
- enum 필드
- VO 필드
- 관계 필드
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (12)
src/main/java/com/gdschongik/gdsc/domain/coupon/domain/IssuedCoupon.java(2 hunks)src/main/java/com/gdschongik/gdsc/domain/member/domain/Member.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/membership/domain/Membership.java(2 hunks)src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java(3 hunks)src/main/java/com/gdschongik/gdsc/domain/recruitment/domain/Recruitment.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRound.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/domain/AssignmentHistory.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/domain/Study.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyAchievement.java(2 hunks)src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyAnnouncement.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyDetail.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyHistory.java(2 hunks)
🔇 Additional comments (16)
src/main/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRound.java (2)
37-39: 필드 선언이 잘 구성되어 있습니다!엔티티 필드 순서 규칙에 맞게 enum 필드가 적절히 배치되었습니다.
62-70: 정적 팩토리 메서드가 적절히 수정되었습니다.매개변수 순서가 생성자와 일치하도록 개선되었으며, recruitment에서 기본 엔티티 필드를 가져오는 방식이 명확합니다.
src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyAchievement.java (2)
33-34: 필드 선언이 적절히 개선되었습니다!
@Enumerated(EnumType.STRING)어노테이션 추가와 필드 순서 변경이 잘 이루어졌습니다. 특히 EnumType을 STRING으로 지정한 것이 데이터 안정성 측면에서 좋은 선택입니다.
45-46: 생성자 시그니처가 일관성 있게 변경되었습니다!필드 순서와 동일하게 파라미터 순서를 변경하여 코드의 일관성이 향상되었습니다.
src/main/java/com/gdschongik/gdsc/domain/recruitment/domain/Recruitment.java (1)
33-37: 생성자 매개변수 순서 변경이 적절합니다.PR 목적에 맞게 필드 순서가 잘 정리되었으며, 부모 클래스 생성자 호출도 올바르게 구현되었습니다.
src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java (1)
Line range hint
91-114: 정적 팩토리 메서드 구현이 잘 되어있습니다.
createPending과createFree메서드가 각각의 상태에 맞게 적절히 구현되어 있으며, 무료 주문에 대한 유효성 검사도 잘 구현되어 있습니다.src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyAnnouncement.java (2)
32-34: 필드 순서 변경이 잘 되었습니다!관계 필드(relationship field)를 엔티티 끝부분으로 이동한 것이 좋은 구조적 개선입니다.
FetchType.LAZY를 사용한 것도 적절합니다.
37-40: 생성자 매개변수 순서 변경에 따른 영향도 확인이 필요합니다.생성자의 매개변수 순서가 변경되었습니다. 이 변경이 다른 코드에 미치는 영향을 확인해야 합니다.
✅ Verification successful
생성자 매개변수 순서 변경은 안전합니다.
빌더 패턴을 통해서만 객체가 생성되고 있어, 생성자의 매개변수 순서 변경이 다른 코드에 영향을 주지 않습니다. 직접적인 생성자 호출이 발견되지 않았으며, 모든 객체 생성이 빌더를 통해 이루어지고 있습니다.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: 생성자 사용 패턴 검색 # StudyAnnouncement 생성자 호출 패턴 검색 rg "new StudyAnnouncement\(" --type javaLength of output: 42
Script:
#!/bin/bash # Description: 다양한 패턴으로 StudyAnnouncement 사용 검색 # 클래스 이름으로 전체 검색 echo "=== StudyAnnouncement 관련 파일 검색 ===" rg "StudyAnnouncement" --type java -l # Builder 패턴 사용 검색 echo -e "\n=== Builder 패턴 사용 검색 ===" rg "StudyAnnouncement\.builder\(\)" --type java -A 5 # 테스트 파일에서 사용 패턴 검색 echo -e "\n=== 테스트 파일에서 사용 검색 ===" fd -e java -g "*Test.java" --exec rg "StudyAnnouncement" {}Length of output: 1764
src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyHistory.java (1)
38-42: 필드 순서가 잘 정리되었습니다!엔티티 필드가 다음과 같은 순서로 잘 정리되어 있습니다:
- ID 필드
- 일반 필드 (repositoryLink)
- Enum 필드 (studyHistoryStatus)
- 관계 필드 (student, study)
src/main/java/com/gdschongik/gdsc/domain/coupon/domain/IssuedCoupon.java (1)
35-39: 필드 순서와 생성자 시그니처가 일관성있게 변경되었습니다!
- 필드가 ID, 일반 필드(hasRevoked, usedAt), 관계 필드(coupon, member) 순으로 정리되었습니다.
- 생성자 매개변수 순서도 필드 순서와 일치하도록 변경되었습니다.
Also applies to: 49-50
src/main/java/com/gdschongik/gdsc/domain/membership/domain/Membership.java (1)
37-39: 필드 순서와 생성자 시그니처가 잘 정리되었습니다!
- 필드가 ID, 값 객체(regularRequirement), 관계 필드(member, recruitmentRound) 순으로 정리되었습니다.
- 생성자 매개변수도 필드 순서와 동일하게 변경되었습니다.
Also applies to: 49-50
src/main/java/com/gdschongik/gdsc/domain/study/domain/StudyDetail.java (1)
64-65: 생성자 매개변수 순서가 적절합니다.필드 순서와 일관성 있게 생성자 매개변수가 정리되었습니다.
src/main/java/com/gdschongik/gdsc/domain/study/domain/Study.java (2)
55-60: 시간 관련 필드가 적절하게 추가되었습니다.시작 시간과 종료 시간 필드가 명확한 주석과 함께 추가되었습니다.
83-92: 생성자와 create 메서드의 매개변수 순서가 일관성 있게 변경되었습니다.필드 순서와 동일한 패턴으로 매개변수가 정리되었으며, 모든 유효성 검사 로직이 잘 유지되고 있습니다.
Also applies to: 107-116
src/main/java/com/gdschongik/gdsc/domain/member/domain/Member.java (2)
42-43: 필드 순서가 체계적으로 정리되었습니다.Id, 일반 필드, enum 필드 순으로 잘 정리되었습니다.
Also applies to: 46-47, 57-70
79-105: 생성자 매개변수 순서가 필드 순서와 일치하도록 잘 변경되었습니다.필드 순서와 동일한 패턴으로 생성자 매개변수가 정리되었습니다.
src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java
Outdated
Show resolved
Hide resolved
| public static StudyAnnouncement create(String title, String link, Study study) { | ||
| return StudyAnnouncement.builder().title(title).link(link).study(study).build(); |
There was a problem hiding this comment.
💡 Codebase verification
팩토리 메서드 호출 순서가 잘못되었습니다.
StudyAnnouncement.create() 메서드는 (title, link, study) 순서로 매개변수를 기대하지만, 다음 위치에서 잘못된 순서로 호출되고 있습니다:
src/main/java/com/gdschongik/gdsc/domain/study/application/MentorStudyService.java에서(study, title, link)순서로 전달하고 있습니다.
매개변수 순서를 StudyAnnouncement.create(request.title(), request.link(), study)로 수정해야 합니다.
🔗 Analysis chain
팩토리 메서드 시그니처 변경에 따른 영향도 확인이 필요합니다.
create 메서드의 매개변수 순서가 변경되었습니다. 이 메서드를 사용하는 모든 곳에서 올바른 순서로 인자를 전달하는지 확인이 필요합니다.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: create 메서드 호출 패턴 검색
# StudyAnnouncement.create 메서드 호출 패턴 검색
rg "StudyAnnouncement\.create\(" --type java
Length of output: 241
| public static StudyDetail create(Long week, String attendanceNumber, Period period, Study study) { | ||
| return StudyDetail.builder() | ||
| .study(study) | ||
| .week(week) | ||
| .period(period) | ||
| .attendanceNumber(attendanceNumber) | ||
| .period(period) | ||
| .curriculum(Curriculum.empty()) | ||
| .assignment(Assignment.empty()) | ||
| .study(study) |
There was a problem hiding this comment.
중복된 period 설정 수정 필요
builder 호출에서 period가 두 번 설정되고 있습니다:
return StudyDetail.builder()
.week(week)
- .period(period)
.attendanceNumber(attendanceNumber)
.period(period)
.curriculum(Curriculum.empty())
.assignment(Assignment.empty())
.study(study)
.build();📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| public static StudyDetail create(Long week, String attendanceNumber, Period period, Study study) { | |
| return StudyDetail.builder() | |
| .study(study) | |
| .week(week) | |
| .period(period) | |
| .attendanceNumber(attendanceNumber) | |
| .period(period) | |
| .curriculum(Curriculum.empty()) | |
| .assignment(Assignment.empty()) | |
| .study(study) | |
| public static StudyDetail create(Long week, String attendanceNumber, Period period, Study study) { | |
| return StudyDetail.builder() | |
| .week(week) | |
| .attendanceNumber(attendanceNumber) | |
| .period(period) | |
| .curriculum(Curriculum.empty()) | |
| .assignment(Assignment.empty()) | |
| .study(study) |
There was a problem hiding this comment.
@uwoobeat 네, 확인해보니 period 필드가 builder에서 두 번 설정되고 있습니다. 첫 번째 .period(period)를 제거하면 됩니다:
public static StudyDetail create(Long week, String attendanceNumber, Period period, Study study) {
return StudyDetail.builder()
.week(week)
.attendanceNumber(attendanceNumber)
.period(period)
.curriculum(Curriculum.empty())
.assignment(Assignment.empty())
.study(study)
.build();
}이렇게 수정하면 중복 설정이 제거되어 코드가 더 깔끔해질 것 같습니다.
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Actionable comments posted: 12
🔭 Outside diff range comments (1)
src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRoundValidatorTest.java (1)
Line range hint
1-233: 코드 포맷팅 위반 사항을 수정해 주세요.GitHub Actions에서 스타일 체크 실패가 보고되었습니다. 다음 사항들을 확인해 주세요:
- 메서드 체이닝 시 일관된 들여쓰기 적용
- 파라미터 나열 시 일관된 들여쓰기 적용
- 테스트 메서드 간 일관된 줄 간격 유지
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
🧹 Nitpick comments (3)
src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRoundValidatorTest.java (1)
28-32: 매개변수 순서가 일관성 있게 변경되었습니다.매개변수 순서가 PR의 목적에 맞게 잘 변경되었으나, 코드 포맷팅 위반이 있습니다. 들여쓰기를 일관되게 적용해 주세요.
- FEE_NAME, FEE, - Period.of(LocalDateTime.of(2025, 3, 2, 0, 0), LocalDateTime.of(2025, 8, 31, 0, 0)), - 2025, - SEMESTER_TYPE + FEE_NAME, FEE, + Period.of(LocalDateTime.of(2025, 3, 2, 0, 0), LocalDateTime.of(2025, 8, 31, 0, 0)), + 2025, + SEMESTER_TYPE🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/helper/FixtureHelper.java (1)
91-94: 매개변수 순서는 좋으나 코드 포맷팅 개선이 필요합니다.매개변수 순서가 엔티티 필드 순서(일반 필드 → 열거형 필드 → 관계 필드)와 일치하도록 잘 변경되었습니다. 다만, 파이프라인 오류를 해결하기 위해 포맷팅 수정이 필요합니다.
다음과 같이 들여쓰기를 수정해주세요:
return Study.create( - STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, - DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR, - SEMESTER_TYPE - ); + STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, + DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR, + SEMESTER_TYPE);🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/coupon/util/CouponNameUtilTest.java (1)
24-28: 매개변수 순서가 엔티티 필드 순서와 일치하도록 변경되었습니다.Study.create() 메서드의 매개변수 순서가 엔티티 필드 순서 표준화 작업과 일치하도록 잘 변경되었습니다. 일반 필드, enum 필드, VO 필드, 관계 필드 순으로 정렬되어 있습니다.
하지만 가독성을 위해 매개변수를 좀 더 논리적인 그룹으로 나누어 정렬하는 것이 좋을 것 같습니다.
다음과 같이 매개변수를 그룹화하여 정렬하는 것을 제안드립니다:
- "기초 백엔드 스터디", TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, - DAY_OF_WEEK, STUDY_ONGOING_PERIOD, - Period.of(START_DATE.minusDays(10), START_DATE.minusDays(5)), mentor, 2025, - SemesterType.FIRST + // 기본 정보 + "기초 백엔드 스터디", + 2025, + SemesterType.FIRST, + // 스터디 일정 + TOTAL_WEEK, + STUDY_START_TIME, + STUDY_END_TIME, + DAY_OF_WEEK, + // 스터디 설정 + ONLINE_STUDY, + STUDY_ONGOING_PERIOD, + Period.of(START_DATE.minusDays(10), START_DATE.minusDays(5)), + // 관계 + mentor🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (18)
src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java(2 hunks)src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java(2 hunks)src/main/java/com/gdschongik/gdsc/domain/recruitment/application/AdminRecruitmentService.java(2 hunks)src/main/java/com/gdschongik/gdsc/domain/study/application/MentorStudyAchievementService.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/application/MentorStudyService.java(1 hunks)src/main/java/com/gdschongik/gdsc/domain/study/factory/StudyDomainFactory.java(2 hunks)src/test/java/com/gdschongik/gdsc/domain/coupon/util/CouponNameUtilTest.java(1 hunks)src/test/java/com/gdschongik/gdsc/domain/member/domain/MemberValidatorTest.java(1 hunks)src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipTest.java(1 hunks)src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipValidatorTest.java(1 hunks)src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java(6 hunks)src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java(6 hunks)src/test/java/com/gdschongik/gdsc/domain/recruitment/application/AdminRecruitmentServiceTest.java(1 hunks)src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRoundValidatorTest.java(11 hunks)src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentTest.java(1 hunks)src/test/java/com/gdschongik/gdsc/domain/study/domain/StudyTest.java(5 hunks)src/test/java/com/gdschongik/gdsc/helper/FixtureHelper.java(3 hunks)src/test/java/com/gdschongik/gdsc/helper/IntegrationTest.java(4 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java
🧰 Additional context used
🪛 GitHub Actions: Check Style and Test to Develop
src/test/java/com/gdschongik/gdsc/domain/coupon/util/CouponNameUtilTest.java
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRoundValidatorTest.java
[error] Code formatting violations present
src/main/java/com/gdschongik/gdsc/domain/study/application/MentorStudyService.java
[error] 123-124: Code formatting violation: Incorrect line breaks in StudyAnnouncement.create() call
src/test/java/com/gdschongik/gdsc/domain/study/domain/StudyTest.java
[error] Code formatting violations present
src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java
[error] 63-64: Code formatting violation: Incorrect line breaks in Order.createPending() call
src/main/java/com/gdschongik/gdsc/domain/recruitment/application/AdminRecruitmentService.java
[error] 42-52: Code formatting violation: Incorrect line breaks and indentation in Recruitment.create() call
src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipValidatorTest.java
[error] Code formatting violations present
src/main/java/com/gdschongik/gdsc/domain/study/factory/StudyDomainFactory.java
[error] 18-33: Code formatting violation: Incorrect line breaks and indentation in Study.create() call
src/test/java/com/gdschongik/gdsc/domain/recruitment/application/AdminRecruitmentServiceTest.java
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/helper/FixtureHelper.java
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/helper/IntegrationTest.java
[error] Code formatting violations present
🔇 Additional comments (26)
src/test/java/com/gdschongik/gdsc/domain/member/domain/MemberValidatorTest.java (2)
28-28: 매개변수 순서가 일관성 있게 변경되었습니다!
Recruitment.create()메서드의 매개변수 순서가 PR의 목적에 맞게 잘 정리되었습니다. 일반 필드(FEE_NAME,FEE)가 먼저 오고, VO(START_TO_END_PERIOD)와 열거형 필드(ACADEMIC_YEAR,SEMESTER_TYPE)가 뒤따르는 구조로 변경되었습니다.
31-31: 매개변수 순서가 체계적으로 개선되었습니다!
RecruitmentRound.create()메서드의 매개변수 순서도 일관된 규칙에 따라 잘 정리되었습니다. 일반 필드(RECRUITMENT_ROUND_NAME), 열거형(ROUND_TYPE), VO(Period), 그리고 관계 필드(recruitment)순으로 배치되어 있습니다.src/main/java/com/gdschongik/gdsc/domain/study/application/MentorStudyService.java (1)
126-127: 매개변수 순서 변경이 적절합니다엔티티 필드 순서 표준화에 맞춰 매개변수 순서가 적절하게 변경되었습니다. 이는 PR의 목적에 부합하며, 코드의 일관성을 향상시킵니다.
src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentRoundValidatorTest.java (1)
78-79: RecruitmentRound 생성 메서드의 매개변수 순서가 일관되게 변경되었습니다.모든 테스트 케이스에서 RecruitmentRound.create() 메서드의 매개변수 순서가 동일한 패턴으로 잘 변경되었습니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/helper/FixtureHelper.java (3)
74-74: 매개변수 순서가 엔티티 필드 순서와 일치하도록 잘 수정되었습니다!일반 필드(FEE_NAME, fee)를 먼저 배치하고, 기간(Period) 및 열거형 필드(academicYear, semesterType)를 뒤로 이동한 것이 PR의 목적과 잘 부합합니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
77-77: RecruitmentRound.create 메서드의 매개변수 순서도 적절하게 변경되었습니다!일반 필드(name, roundType)를 먼저 배치하고, 기간(Period) 및 관계 필드(recruitment)를 뒤로 이동한 구조가 일관성 있습니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
103-103: StudyDetail 생성 메서드들의 매개변수 순서가 일관되게 변경되었습니다!두 메서드 모두 일반 필드를 먼저 배치하고 관계 필드(study)를 마지막에 배치하여, PR의 목적인 엔티티 필드 순서 표준화가 잘 적용되었습니다.
Also applies to: 108-109
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/main/java/com/gdschongik/gdsc/domain/study/factory/StudyDomainFactory.java (1)
37-37: 매개변수 순서 변경이 일관성 있게 적용됨StudyDetail.create() 메서드의 매개변수 순서가 PR의 목적에 맞게 일관성 있게 변경되었습니다. 엔티티 필드 순서 변경 규칙이 잘 반영되었습니다.
src/main/java/com/gdschongik/gdsc/domain/study/application/MentorStudyAchievementService.java (1)
55-55: 매개변수 순서 변경이 적절하게 이루어짐StudyAchievement.create() 메서드의 매개변수 순서가 엔티티 필드 순서 변경 규칙에 맞게 변경되었습니다. achievementType(enum), member(일반 필드), study(관계 필드) 순으로 정렬되어 일관성이 유지되었습니다.
src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipValidatorTest.java (1)
44-46: 매개변수 순서 변경이 일관성 있게 적용되었습니다.매개변수 순서가 엔티티 필드 순서와 일치하도록 잘 변경되었습니다. 이는 코드의 가독성과 유지보수성을 향상시킵니다.
다만, 코드 스타일 검사에서 오류가 발생했습니다. 다음 사항을 확인해 주세요:
- 들여쓰기가 올바른지 확인
- 줄 바꿈이 일관되게 적용되었는지 확인
✅ Verification successful
매개변수 순서 변경이 올바르게 적용되었으며, 코드 스타일도 정확합니다.
코드를 검토한 결과, 다음 사항들이 모두 적절하게 구현되었습니다:
- 매개변수 순서가 엔티티 필드 순서와 일치하도록 잘 정렬되어 있습니다
- 들여쓰기와 줄 바꿈이 일관되게 적용되어 있습니다
- 전반적인 코드 포맷팅이 프로젝트 스타일 가이드를 잘 따르고 있습니다
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check code style violations # Note: Using grep to find indentation issues in the changed lines rg -U "Recruitment\.create\([\s\S]*?\)" | grep -P "^[ ]{16}" rg -U "RecruitmentRound\.create\([\s\S]*?\)" | grep -P "^[ ]{16}"Length of output: 130
Script:
#!/bin/bash # Get the exact content of the file cat src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipValidatorTest.java # Check indentation patterns specifically around the changed lines rg -A 2 -B 2 "FEE_NAME, fee," src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipValidatorTest.javaLength of output: 4545
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java (2)
61-61: 메서드 호출의 매개변수 순서가 일관되게 변경되었습니다.
Order.createFree메서드의 매개변수 순서가 PR의 목적에 맞게 적절하게 변경되었습니다.🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
81-83: 예외 테스트의 매개변수 순서도 일관되게 변경되었습니다.예외를 검증하는 테스트 케이스에서도 매개변수 순서가 올바르게 조정되었습니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java (2)
66-67: 메서드 매개변수 순서 변경이 서비스 계층에도 일관되게 적용되었습니다.
createPendingOrder메서드에서Order.createPending호출 시 매개변수 순서가 적절하게 변경되었습니다.하지만 들여쓰기 포맷팅을 수정해야 합니다:
- Order order = Order.createPending(request.orderNanoId(), moneyInfo, - membership, issuedCoupon); + Order order = Order.createPending(request.orderNanoId(), + moneyInfo, + membership, + issuedCoupon);
172-172: 무료 주문 생성 메서드도 일관된 매개변수 순서를 따르고 있습니다.
createFreeOrder메서드에서Order.createFree호출의 매개변수 순서가 올바르게 변경되었습니다.src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java (1)
297-297: 검증 테스트의 메서드 호출도 일관된 매개변수 순서로 변경되었습니다.
OrderValidatorTest의 모든 테스트 케이스에서Order.createPending과Order.createFree메서드 호출의 매개변수 순서가 일관되게 변경되었습니다.다만, 다른 파일들과 마찬가지로 포맷팅 이슈가 있습니다:
- Order order = Order.createPending( - "nanoId", MoneyInfo.of(MONEY_20000_WON, Money.ZERO, MONEY_20000_WON), membership, null); + Order order = Order.createPending("nanoId", + MoneyInfo.of(MONEY_20000_WON, Money.ZERO, MONEY_20000_WON), + membership, + null);Also applies to: 325-326, 353-354, 379-379, 403-403, 428-429
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/coupon/util/CouponNameUtilTest.java (1)
Line range hint
1-35: 코드 포맷팅 위반 사항을 수정해주세요.GitHub Actions에서 스타일 체크 실패가 보고되었습니다. 전체 파일의 포맷팅을 검토하고 수정해주시기 바랍니다.
다음 스크립트를 실행하여 구체적인 포맷팅 위반 사항을 확인할 수 있습니다:
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/test/java/com/gdschongik/gdsc/domain/recruitment/domain/RecruitmentTest.java (1)
22-22: 매개변수 순서 변경이 적절합니다!기본 필드(FEE_NAME, FEE)를 먼저 배치하고, 복잡한 객체(Period)와 열거형 필드(ACADEMIC_YEAR, SEMESTER_TYPE)를 뒤에 배치한 것이 더 논리적입니다.
src/test/java/com/gdschongik/gdsc/domain/membership/domain/MembershipTest.java (1)
32-34: 매개변수 순서 변경이 일관성 있게 적용되었습니다!
Recruitment.create()와RecruitmentRound.create()모두 동일한 패턴으로 매개변수 순서가 변경되어 일관성이 잘 유지되었습니다.src/test/java/com/gdschongik/gdsc/domain/recruitment/application/AdminRecruitmentServiceTest.java (1)
74-75: 매개변수 순서 변경이 테스트 코드에도 잘 반영되었습니다!서비스 계층의 변경사항이 테스트 코드에도 일관성 있게 적용되어 있습니다.
Also applies to: 79-79
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
src/main/java/com/gdschongik/gdsc/domain/recruitment/application/AdminRecruitmentService.java (1)
84-84: 매개변수 순서 변경이 서비스 계층에도 잘 반영되었습니다!
RecruitmentRound.create()메서드의 매개변수 순서가 일관성 있게 변경되었습니다.src/test/java/com/gdschongik/gdsc/helper/IntegrationTest.java (6)
191-191: 매개변수 순서 변경이 적절합니다엔티티 필드 순서에 맞춰 매개변수가 잘 정렬되었습니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
207-207: 오버로드된 메서드의 매개변수 순서가 일관성 있게 변경되었습니다기본 메서드와 동일한 패턴으로 매개변수가 정렬되어 있습니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
213-213: 엔티티 필드 구조에 맞게 잘 정렬되었습니다
Recruitment생성 메서드의 매개변수 순서가 엔티티 필드 구조를 잘 반영하고 있습니다.🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
250-251: 매개변수 순서가 적절히 변경되었습니다
StudyDetail생성 메서드의 매개변수 순서가 엔티티 필드 구조를 잘 반영하고 있습니다.🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
256-257: 오버로드된 메서드의 매개변수 순서가 일관성 있게 변경되었습니다기본 메서드와 동일한 패턴으로 매개변수가 정렬되어 있습니다.
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
267-267: 매개변수 순서 변경이 적절합니다
StudyAchievement생성 메서드의 매개변수 순서가 엔티티 필드 구조를 잘 반영하고 있습니다.🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| StudyAnnouncement studyAnnouncement = StudyAnnouncement.create(request.title(), | ||
| request.link(), study); |
There was a problem hiding this comment.
코드 포맷팅 수정 필요
코드 스타일 검사에서 지적된 StudyAnnouncement.create() 호출의 줄바꿈을 수정해야 합니다.
다음과 같이 수정하세요:
- StudyAnnouncement studyAnnouncement = StudyAnnouncement.create(request.title(),
- request.link(), study);
+ StudyAnnouncement studyAnnouncement = StudyAnnouncement.create(
+ request.title(), request.link(), study);📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| StudyAnnouncement studyAnnouncement = StudyAnnouncement.create(request.title(), | |
| request.link(), study); | |
| StudyAnnouncement studyAnnouncement = StudyAnnouncement.create( | |
| request.title(), request.link(), study); |
| request.title(), request.totalWeek(), request.studyStartTime(), | ||
| request.studyEndTime(), request.studyType(), request.dayOfWeek(), | ||
| Period.of(request.startDate().atStartOfDay(), endDate.atTime(LocalTime.MAX)), Period.of( | ||
| request.applicationStartDate().atStartOfDay(), | ||
| request.applicationEndDate().atTime(LocalTime.MAX)), | ||
| request.totalWeek(), | ||
| request.studyType(), | ||
| request.dayOfWeek(), | ||
| request.studyStartTime(), | ||
| request.studyEndTime()); | ||
| request.applicationEndDate().atTime(LocalTime.MAX)), mentor, request.academicYear(), | ||
| request.semesterType() | ||
| ); |
There was a problem hiding this comment.
코드 스타일 위반 수정 필요
GitHub Actions에서 보고된 코드 스타일 위반을 수정해야 합니다. Study.create() 메서드 호출의 줄 바꿈과 들여쓰기가 올바르지 않습니다.
다음과 같이 수정하는 것을 제안합니다:
return Study.create(
- request.title(), request.totalWeek(), request.studyStartTime(),
- request.studyEndTime(), request.studyType(), request.dayOfWeek(),
- Period.of(request.startDate().atStartOfDay(), endDate.atTime(LocalTime.MAX)), Period.of(
- request.applicationStartDate().atStartOfDay(),
- request.applicationEndDate().atTime(LocalTime.MAX)), mentor, request.academicYear(),
- request.semesterType()
+ request.title(),
+ request.totalWeek(),
+ request.studyStartTime(),
+ request.studyEndTime(),
+ request.studyType(),
+ request.dayOfWeek(),
+ Period.of(request.startDate().atStartOfDay(), endDate.atTime(LocalTime.MAX)),
+ Period.of(
+ request.applicationStartDate().atStartOfDay(),
+ request.applicationEndDate().atTime(LocalTime.MAX)
+ ),
+ mentor,
+ request.academicYear(),
+ request.semesterType()
);📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| request.title(), request.totalWeek(), request.studyStartTime(), | |
| request.studyEndTime(), request.studyType(), request.dayOfWeek(), | |
| Period.of(request.startDate().atStartOfDay(), endDate.atTime(LocalTime.MAX)), Period.of( | |
| request.applicationStartDate().atStartOfDay(), | |
| request.applicationEndDate().atTime(LocalTime.MAX)), | |
| request.totalWeek(), | |
| request.studyType(), | |
| request.dayOfWeek(), | |
| request.studyStartTime(), | |
| request.studyEndTime()); | |
| request.applicationEndDate().atTime(LocalTime.MAX)), mentor, request.academicYear(), | |
| request.semesterType() | |
| ); | |
| return Study.create( | |
| request.title(), | |
| request.totalWeek(), | |
| request.studyStartTime(), | |
| request.studyEndTime(), | |
| request.studyType(), | |
| request.dayOfWeek(), | |
| Period.of(request.startDate().atStartOfDay(), endDate.atTime(LocalTime.MAX)), | |
| Period.of( | |
| request.applicationStartDate().atStartOfDay(), | |
| request.applicationEndDate().atTime(LocalTime.MAX) | |
| ), | |
| mentor, | |
| request.academicYear(), | |
| request.semesterType() | |
| ); |
| "testNanoId", MoneyInfo.of(MONEY_20000_WON, MONEY_5000_WON, MONEY_15000_WON), | ||
| membership, null); |
There was a problem hiding this comment.
동일한 포맷팅 이슈가 여러 곳에서 발견되었습니다.
위에서 제안한 것과 같은 포맷팅 이슈가 다음 라인들에서도 발견되었습니다. 일관성 있게 수정이 필요합니다.
모든 Order.createPending과 Order.createFree 메서드 호출에 동일한 포맷팅 규칙을 적용해 주세요.
Also applies to: 153-153, 176-177, 297-297, 325-326, 353-354, 379-379, 403-403, 428-429
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| "testNanoId", MoneyInfo.of(MONEY_20000_WON, MONEY_5000_WON, MONEY_15000_WON), | ||
| membership, null); |
There was a problem hiding this comment.
들여쓰기 포맷팅 이슈가 있습니다.
파이프라인 실패 로그에 따르면 코드 포맷팅 위반이 있습니다. 메서드 호출의 들여쓰기를 수정해 주세요.
다음과 같이 수정하는 것을 제안합니다:
- Order order = Order.createPending(
- "testNanoId", MoneyInfo.of(MONEY_20000_WON, MONEY_5000_WON, MONEY_15000_WON),
- membership, null);
+ Order order = Order.createPending("testNanoId",
+ MoneyInfo.of(MONEY_20000_WON, MONEY_5000_WON, MONEY_15000_WON),
+ membership,
+ null);📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "testNanoId", MoneyInfo.of(MONEY_20000_WON, MONEY_5000_WON, MONEY_15000_WON), | |
| membership, null); | |
| Order order = Order.createPending("testNanoId", | |
| MoneyInfo.of(MONEY_20000_WON, MONEY_5000_WON, MONEY_15000_WON), | |
| membership, | |
| null); |
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| request.feeName(), Money.from(request.fee()), | ||
| Period.of(request.semesterStartDate(), request.semesterEndDate()), request.academicYear(), | ||
| request.semesterType() | ||
| ); |
There was a problem hiding this comment.
코드 포맷팅을 개선해 주세요.
Recruitment.create() 호출의 들여쓰기와 줄바꿈이 일관성이 없습니다. 다음과 같이 수정하는 것을 제안합니다:
- Recruitment recruitment = Recruitment.create(
- request.feeName(), Money.from(request.fee()),
- Period.of(request.semesterStartDate(), request.semesterEndDate()), request.academicYear(),
- request.semesterType()
- );
+ Recruitment recruitment = Recruitment.create(
+ request.feeName(),
+ Money.from(request.fee()),
+ Period.of(request.semesterStartDate(), request.semesterEndDate()),
+ request.academicYear(),
+ request.semesterType());📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| request.feeName(), Money.from(request.fee()), | |
| Period.of(request.semesterStartDate(), request.semesterEndDate()), request.academicYear(), | |
| request.semesterType() | |
| ); | |
| request.feeName(), | |
| Money.from(request.fee()), | |
| Period.of(request.semesterStartDate(), request.semesterEndDate()), | |
| request.academicYear(), | |
| request.semesterType()); |
| STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, | ||
| DAY_OF_WEEK, START_TO_END_PERIOD, applicationPeriod, guestMember, ACADEMIC_YEAR, | ||
| SEMESTER_TYPE | ||
| )) |
There was a problem hiding this comment.
들여쓰기 스타일 수정 필요
코드 포맷팅 검사 실패의 원인이 되는 들여쓰기 불일치가 있습니다. 파라미터 정렬을 다음과 같이 수정해주세요:
- STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY,
- DAY_OF_WEEK, START_TO_END_PERIOD, applicationPeriod, guestMember, ACADEMIC_YEAR,
- SEMESTER_TYPE
+ STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY,
+ DAY_OF_WEEK, START_TO_END_PERIOD, applicationPeriod, guestMember, ACADEMIC_YEAR,
+ SEMESTER_TYPE📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, | |
| DAY_OF_WEEK, START_TO_END_PERIOD, applicationPeriod, guestMember, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| )) | |
| STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, | |
| DAY_OF_WEEK, START_TO_END_PERIOD, applicationPeriod, guestMember, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| )) |
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| STUDY_TITLE, TOTAL_WEEK, null, null, ONLINE_STUDY, DAY_OF_WEEK, period, | ||
| applicationPeriod, member, ACADEMIC_YEAR, | ||
| SEMESTER_TYPE | ||
| )) |
There was a problem hiding this comment.
들여쓰기 패턴 통일 필요
테스트 케이스 전반에 걸쳐 일관된 들여쓰기를 적용해주세요:
- STUDY_TITLE, TOTAL_WEEK, null, null, ONLINE_STUDY, DAY_OF_WEEK, period,
- applicationPeriod, member, ACADEMIC_YEAR,
- SEMESTER_TYPE
+ STUDY_TITLE, TOTAL_WEEK, null, null, ONLINE_STUDY, DAY_OF_WEEK, period,
+ applicationPeriod, member, ACADEMIC_YEAR,
+ SEMESTER_TYPE📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| STUDY_TITLE, TOTAL_WEEK, null, null, ONLINE_STUDY, DAY_OF_WEEK, period, | |
| applicationPeriod, member, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| )) | |
| STUDY_TITLE, TOTAL_WEEK, null, null, ONLINE_STUDY, DAY_OF_WEEK, period, | |
| applicationPeriod, member, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| )) |
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| STUDY_TITLE, TOTAL_WEEK, studyStartTime, studyEndTime, ASSIGNMENT_STUDY, | ||
| DAY_OF_WEEK, period, applicationPeriod, member, ACADEMIC_YEAR, | ||
| SEMESTER_TYPE | ||
| )) |
There was a problem hiding this comment.
마지막 테스트 케이스 들여쓰기 수정 필요
파일의 마지막 테스트 케이스에서도 동일한 들여쓰기 문제가 발견되었습니다:
- STUDY_TITLE, TOTAL_WEEK, studyStartTime, studyEndTime, ASSIGNMENT_STUDY,
- DAY_OF_WEEK, period, applicationPeriod, member, ACADEMIC_YEAR,
- SEMESTER_TYPE
+ STUDY_TITLE, TOTAL_WEEK, studyStartTime, studyEndTime, ASSIGNMENT_STUDY,
+ DAY_OF_WEEK, period, applicationPeriod, member, ACADEMIC_YEAR,
+ SEMESTER_TYPE📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| STUDY_TITLE, TOTAL_WEEK, studyStartTime, studyEndTime, ASSIGNMENT_STUDY, | |
| DAY_OF_WEEK, period, applicationPeriod, member, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| )) | |
| STUDY_TITLE, TOTAL_WEEK, studyStartTime, studyEndTime, ASSIGNMENT_STUDY, | |
| DAY_OF_WEEK, period, applicationPeriod, member, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| )) |
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| STUDY_TITLE, totalWeek, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, DAY_OF_WEEK, | ||
| period, applicationPeriod, mentor, ACADEMIC_YEAR, | ||
| SEMESTER_TYPE | ||
| ); |
There was a problem hiding this comment.
들여쓰기 수준 조정 필요
현재 들여쓰기를 다음과 같이 수정해주세요:
- STUDY_TITLE, totalWeek, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, DAY_OF_WEEK,
- period, applicationPeriod, mentor, ACADEMIC_YEAR,
- SEMESTER_TYPE
+ STUDY_TITLE, totalWeek, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, DAY_OF_WEEK,
+ period, applicationPeriod, mentor, ACADEMIC_YEAR,
+ SEMESTER_TYPE📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| STUDY_TITLE, totalWeek, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, DAY_OF_WEEK, | |
| period, applicationPeriod, mentor, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| ); | |
| STUDY_TITLE, totalWeek, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, DAY_OF_WEEK, | |
| period, applicationPeriod, mentor, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| ); |
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
| STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, | ||
| DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR, | ||
| SEMESTER_TYPE | ||
| ); |
There was a problem hiding this comment.
들여쓰기 개선 필요
매개변수 순서는 적절하나, 들여쓰기를 다음과 같이 수정해주세요:
- STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY,
- DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR,
- SEMESTER_TYPE
+ STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY,
+ DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR,
+ SEMESTER_TYPE📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, | |
| DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| ); | |
| STUDY_TITLE, TOTAL_WEEK, STUDY_START_TIME, STUDY_END_TIME, ONLINE_STUDY, | |
| DAY_OF_WEEK, period, applicationPeriod, mentor, ACADEMIC_YEAR, | |
| SEMESTER_TYPE | |
| ); |
🧰 Tools
🪛 GitHub Actions: Check Style and Test to Develop
[error] Code formatting violations present
|
이슈에 일반 필드에 대한 기준은 따로 명시되어 있지 않은 것 같은데, 혹시 어떤 부분인지 알려주실 수 있나요? 'id -> status / type enum -> 일반 필드 -> vo -> 연관 엔티티' 인지 |
사실 모든 케이스 대응하도록 규칙 만드는 건 어려워서, 이정도로만 보고 나머지는 리뷰로 조율해가면서 맞춰봐야 할 것 같네요 1 .id -> status / type enum -> 그 외 일반 필드 / vo / enum -> 연관 엔티티 순서로 한다 |
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
| public static StudyDetail create(Long week, String attendanceNumber, Period period, Study study) { | ||
| return StudyDetail.builder() | ||
| .study(study) | ||
| .week(week) | ||
| .period(period) | ||
| .attendanceNumber(attendanceNumber) | ||
| .period(period) | ||
| .curriculum(Curriculum.empty()) | ||
| .assignment(Assignment.empty()) | ||
| .study(study) |
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
📚 기타
Summary by CodeRabbit
릴리즈 노트
도메인 변경
기능 변경
이번 업데이트는 시스템의 내부 구조를 최적화하고 코드의 일관성을 높이는 데 중점을 두었습니다.