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

[BE] chore: H2를 테스트에서만 사용하도록 설정 #1010

Merged
merged 3 commits into from
Dec 18, 2024

Conversation

donghoony
Copy link
Contributor

@donghoony donghoony commented Dec 15, 2024


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

  • 기능 구현한 사항은 없습니다.

🔥 어떻게 해결했나요 ?

  • 각각의 외부 의존성에 버전을 명시했습니다.
  • openapi-starter-webmvc를 제거했습니다 (swagger의 잔재 ㅋㅋ)

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

  • 불필요한 의존성이 있는지만 확인해주세요!

📚 참고 자료, 할 말

  • Springboot 관련 의존성은 plugin으로 되어 따로 작성하지 않는 것이 권장됩니다. 이곳을 확인해 주세요

Each release of Spring Boot is associated with a base version of the Spring Framework. We highly recommend that you do not specify its version.

Copy link

github-actions bot commented Dec 15, 2024

Test Results

157 tests   154 ✅  4s ⏱️
 58 suites    3 💤
 58 files      0 ❌

Results for commit d9048d7.

♻️ This comment has been updated with latest results.

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.

왜 이걸 지금 알았을까요^^ 머쓱..

그런데 일부 라이브러리 버전은 왜 명시하도록 바뀐건가요?
그리고 나머지 라이브러리에는 버전을 명시하지 않은 이유가 있나요?

@donghoony
Copy link
Contributor Author

donghoony commented Dec 15, 2024

그런데 일부 라이브러리 버전은 왜 명시하도록 바뀐건가요? 그리고 나머지 라이브러리에는 버전을 명시하지 않은 이유가 있나요?

참고 자료, 할 말을 확인해주세용~
버전을 명시한 이유는 latest로 자동 설정되므로 코드상의 충돌이 발생할 염려가 있어 그렇습니다. 특히 메이저 버전의 차이가 발생하면 gradle build만으로 코드가 안 돌아갈 수 있어서요~

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.

외부 의존성 역시 스프링이 호환되는 버전을 자동으로 관리해주고 있는 것으로 알아서 버전 명시가 없어도 될 것 같아요~
https://docs.spring.io/spring-boot/appendix/dependency-versions/coordinates.html

@donghoony donghoony changed the title [BE] chore: H2를 테스트에서만 사용하도록 설정, 외부 의존성 버전 명시 [BE] chore: H2를 테스트에서만 사용하도록 설정 Dec 16, 2024
@donghoony donghoony merged commit 69b507a into develop Dec 18, 2024
5 checks passed
@donghoony donghoony deleted the be/chore/1009-h2-testruntimeonly branch December 18, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] H2 데이터베이스를 테스트 환경에서만 사용하도록 변경
4 participants