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

feat: hello 출력 기능 추가 (#4) #5

Merged
merged 4 commits into from
Oct 13, 2024
Merged

feat: hello 출력 기능 추가 (#4) #5

merged 4 commits into from
Oct 13, 2024

Conversation

kamser0415
Copy link
Collaborator

#️⃣ 연관된 이슈

Closes #4

📝 작업 내용

  • hello 기능 추가

✒️ 코드 변경 이유

  • 사용자에게 환영 메시지를 제공하기 위해 hello 기능을 추가하였습니다.

💬 리뷰 요구사항(선택)

@kamser0415 kamser0415 requested a review from f-lab-jd July 12, 2024 13:46
@kamser0415 kamser0415 self-assigned this Jul 12, 2024
@kamser0415 kamser0415 changed the title Feature/#4 hello feat: hello 출력 기능 추가 (#4) Jul 12, 2024
HELP.md Outdated
@@ -0,0 +1,22 @@
# Getting Started
Copy link
Collaborator

Choose a reason for hiding this comment

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

프로젝트와 협업을 위해 필요한 문서일까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

필요한 문서는 아닙니다.

@@ -0,0 +1 @@
shop.sendbox.sendbox.SendboxApplication
Copy link
Collaborator

Choose a reason for hiding this comment

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

build 디렉토리까지 remote 에 올라가는게 필요할까요?
gitignore 정책을 전반적으로 검토부탁드립니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

맞습니다 !
수정하겠습니다

@kamser0415 kamser0415 requested a review from f-lab-jd July 13, 2024 02:03
dependencies {
implementation 'org.springframework.boot:spring-boot-starter-web'
testImplementation 'org.springframework.boot:spring-boot-starter-test'
testRuntimeOnly 'org.junit.platform:junit-platform-launcher'
Copy link
Collaborator

Choose a reason for hiding this comment

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

spring-boot-starter-test 와 별개로 해당 라이브러리를 추가하신 이유는 무엇인가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

implementation 'org.springframework.boot:spring-boot-starter-web'
이 라이브러리를 추가한 이유는 내장 톰캣 사용과 웹 어플리케이션을 개발할 때 필요한 여러 기능을 제공하기 때문입니다.

testImplementation 'org.springframework.boot:spring-boot-starter-test'
이 프레임워크는 Junit 테스트와 기타 테스트를 하기 위한 라이브러리로 앞으로 기능을 개발하기 위함입니다.

testRuntimeOnly 'org.junit.platform:junit-platform-launcher'
Spring Boot 애플리케이션에서 JUnit 5 테스트를 실행하려면 JUnit 5 플랫폼 런처가 필요합니다.
이 런처를 추가하면 테스트 클래스를 찾고 실행할 수 있습니다

Copy link
Collaborator

Choose a reason for hiding this comment

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

starter-test 에도 JUnit 5 가 포함되어있지 않나요~?
https://mvnrepository.com/artifact/org.springframework.boot/spring-boot-starter-test/3.3.1

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

testRuntimeOnly 'org.junit.platform:junit-platform-launcher'가 추가된 이유입니다.

IDE 충돌 방지: IntelliJ IDEA와 같은 IDE에서 JUnit의 버전 충돌 문제를 방지하기 위함입니다. IDE에 번들된 JUnit 버전과 프로젝트에서 사용하는 JUnit 버전이 다를 때 발생할 수 있는 충돌을 예방합니다.

Gradle 버전 변경에 의한 호환성: Gradle 9에서 테스트 프레임워크 구현 의존성의 자동 로딩이 더 이상 지원되지 않기 때문에, 이를 명시적으로 선언하여 Gradle 9에서도 호환성을 유지합니다.

이 두 가지 이유로 testRuntimeOnly 'org.junit.platform:junit-platform-launcher' 의존성을 추가합니다.

참고

spring-io/initializr#1476
https://junit.org/junit5/docs/current

@kamser0415 kamser0415 requested a review from f-lab-jd July 14, 2024 01:33
@kamser0415 kamser0415 merged commit 8339bb9 into main Oct 13, 2024
1 check passed
@kamser0415 kamser0415 deleted the feature/#4-hello branch October 13, 2024 10:12
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.

[enhancement] Hello를 화면 출력하는 기능 추가
2 participants