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.

Sign up for GitHub

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

Jump to bottom

[BE] chore: 롬복 설정에서 불필요한 부분 제거 #1067

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

[BE] chore: 롬복 설정에서 불필요한 부분 제거 #1067

wants to merge 2 commits into from

Conversation

Kimprodp
Copy link
Contributor

🚀 어떤 기능을 구현했나요 ?

  • 롬복은 컴파일 환경에서 annotationProcessor에 의해 설정되므로, testImplementation은 불필요합니다.
  • 추가로 compileOnly가 annotationProcessor를 상속받도록 되어있는데, 이부분은 롬복에서만 사용되고 있습니다. test관련 configuration과 통일성을 갖게하기 위해서 상속방식을 사용하지 않는 것으로 변경했습니다.

🔥 어떻게 해결했나요 ?

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 롬복이 런타임 패키징에 포함되는 것이 치명적이지는 않지만, 불필요한 부분이라 생각되어 변경했습니다.

📚 참고 자료, 할 말

@Kimprodp Kimprodp added this to the 7차 스프린트 milestone Jan 21, 2025
@Kimprodp Kimprodp self-assigned this Jan 21, 2025
@Kimprodp Kimprodp linked an issue Jan 21, 2025 that may be closed by this pull request
[BE] Gradle 설정 파일 내 불필요한 부분을 수정한다. #1066
Open
@Kimprodp Kimprodp changed the title [BE] 롬복 설정에서 불필요한 부분 제거 [BE] chore: 롬복 설정에서 불필요한 부분 제거 Jan 21, 2025
@github-actions GitHub Actions
Copy link

Test Results

165 tests   162 ✅  4s ⏱️
 60 suites    3 💤
 60 files      0 ❌

Results for commit 6ef3111.

nayonsoso
nayonsoso approved these changes Jan 21, 2025
View reviewed changes
Copy link
Contributor

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

와우 이런 부분까지 캐치하다니 gradle 달인이 되었구만유🤩
저도 달인이 되고 싶어요.. 테드한테 많이 배웁니다~!

skylar1220
skylar1220 approved these changes Jan 22, 2025
View reviewed changes
Copy link
Contributor

@skylar1220 skylar1220 left a comment

Choose a reason for hiding this comment

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

롬복은 런타임에서 필요없다. 컴파일에서만 필요하다. annotationProcessor는 별도로 필요하다. 그래서 필요에 의한것만 명시하게 정리했다. 이해했습니다! 굳굳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skylar1220 skylar1220 skylar1220 approved these changes

@nayonsoso nayonsoso nayonsoso approved these changes

@donghoony donghoony donghoony approved these changes

Assignees

@Kimprodp Kimprodp

Labels
✨ BE 🍰 초간단
Projects
2024-review-me
Status: No status
Milestone
7차 스프린트
Development

Successfully merging this pull request may close these issues.

[BE] Gradle 설정 파일 내 불필요한 부분을 수정한다.
4 participants
@Kimprodp @donghoony @nayonsoso @skylar1220