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

[Chore][#5] project setup #6

Merged
merged 4 commits into from
Jan 26, 2025
Merged

Conversation

Stark-Industries0417
Copy link
Collaborator

  • resolve [CHORE] 프로젝트 초기 설정 #5

  • Gradle 멀티모듈 구성:

    • root-project: 루트 프로젝트로 공통 설정 관리
    • game-server: Spring WebFlux 기반 게임 서버 모듈
    • logging-server: Spring Boot 기반 로깅 서버 모듈
  • Kotlin JVM Toolchain 설정:

    • 모든 모듈에서 JVM 17 사용

@Stark-Industries0417 Stark-Industries0417 added 👀 s.needs-review 코드 리뷰가 필요한 상태 ✅ s.completed 작업이 완료된 상태 labels Jan 24, 2025
@Stark-Industries0417 Stark-Industries0417 self-assigned this Jan 24, 2025
@Stark-Industries0417 Stark-Industries0417 changed the title Chore/5 project setup [Chore][#5] project setup Jan 24, 2025
Copy link

@f-lab-saponin f-lab-saponin left a comment

Choose a reason for hiding this comment

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

게임서버와 로깅서버가 완전 다른종류의 프로젝트라, 멀티모듈로 함께있는게 맞을지 고민이 되는 부분이 있는데요.
(제가 이해한 로깅서버는 다른 애플리케이션에도 적용가능한 범용적인 애플리케이션)
위 같은 관점에서는 두 모듈이 공통모듈(혹은 루트프로젝트)을 공유한다는게 좀 어색하게 느껴지는데요.
어떤 것을 공통으로 사용하는 건가요 혹시?

추가로 드는 생각은 멀티모듈에 게임서버, 로깅서버를 같이 포함하는건
게임서버가 샘플용서버라고 가정하면 그렇게 해봐도 되지 않을까 싶네요.

kotlin("jvm") version "2.0.21"
}

subprojects {

Choose a reason for hiding this comment

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

루트 프로젝트에 무엇을 하실 의도이신것 같은데,
루트 + 하위 프로젝트에서 관리하는건 allprojects 로 해봐도 좋을 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

루트 프로젝트에서 별도로 처리할 작업은 없습니다
다만 하위 프로젝트 간에 공통 gradle 설정을 공유하고 하나의 IDE에서 여러 모듈을 편리하게 개발하고자
멀티 모듈로 구성하였습니다

Choose a reason for hiding this comment

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

넵넵 확인했습니다.

@@ -0,0 +1,7 @@
plugins {
id("org.gradle.toolchains.foojay-resolver-convention") version "0.8.0"

Choose a reason for hiding this comment

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

이건 gradle 과 호환되는 JDK 를 자동으로 다운로드해주는 것 같은데,
로컬과 서버가 환경이 달라질 수 도 있을 것 같은 문제가 보이네요.
혹시 다른 장점같은게 있을까요?

Copy link
Collaborator Author

@Stark-Industries0417 Stark-Industries0417 Jan 25, 2025

Choose a reason for hiding this comment

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

�서로 다른 os 환경에서도 적합한 JDK를 설치해주기에 사용하기 적합하다고 판단하여 사용하였습니다!

Copy link

@f-lab-saponin f-lab-saponin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 승인드립니다 👍

로깅 서버에서 구체적으로 어떤 역할을 할지 설계하는것도 필요해보이는데요.
현재 게임서버를 만드는 것을 우선적으로 하고있다면 그부분을 먼저 다해주시면 좋을 것 같아요.
추후에 고민해봅시다!

@Stark-Industries0417 Stark-Industries0417 merged commit 481a0f1 into main Jan 26, 2025
2 checks passed
@Stark-Industries0417 Stark-Industries0417 deleted the chore/5-project-setup branch January 26, 2025 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ s.completed 작업이 완료된 상태 👀 s.needs-review 코드 리뷰가 필요한 상태
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CHORE] 프로젝트 초기 설정
2 participants