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/#107 Maven 배포 구성 및 의존성 추가 #110

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Conversation

miiiinju1
Copy link
Member

🚀 작업 내용

  1. Maven 배포 구성:

    • maven-publish 플러그인을 추가하여 배포 기능을 활성화했습니다.
    • 프로젝트의 Java 컴포넌트를 포함하는 mavenJava라는 배포 구성을 추가했습니다.
    • 빌드된 아티팩트를 build/java 디렉토리에 저장하는 로컬 Maven 리포지토리(local-repo)를 설정했습니다.
  2. Java Toolchain:

    • 프로젝트의 언어 버전을 Java 17로 설정하여 최신 Java 기능과의 호환성을 보장했습니다.
  3. 의존성 추가:

    • 로깅 기능을 위한 logback-classic 의존성을 추가했습니다.
    • SLF4J 로깅 퍼사드를 사용하기 위해 slf4j-api를 추가했습니다.
    • JSON 직렬화 및 역직렬화를 위한 jackson-databind 의존성을 포함했습니다.

📸 이슈 번호

👀 Focus Commits [Optional]

  • 커밋해시: 내용

✍ 궁금한 점

  • 중점적으로 봐줄 내용
  • 변수명 괜찮나요
  • 로직이 좀 더럽나요
  • 가독성이 좀 그런가요?

- fatJar로 jar를 만들도록 수정했습니다.
@miiiinju1 miiiinju1 requested review from tidavid1 and LuizyHub August 22, 2024 08:21
@miiiinju1 miiiinju1 self-assigned this Aug 22, 2024
Copy link
Member

@tidavid1 tidavid1 left a comment

Choose a reason for hiding this comment

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

👍🏻

Copy link
Member

@LuizyHub LuizyHub left a comment

Choose a reason for hiding this comment

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

배포 까지 확인했습니다!

@miiiinju1 miiiinju1 merged commit b84f241 into dev Aug 22, 2024
4 checks passed
@miiiinju1 miiiinju1 deleted the feat/#107 branch August 22, 2024 08:29
LuizyHub pushed a commit that referenced this pull request Aug 22, 2024
* config: maven publish용 gradle 설정 변경

- fatJar로 jar를 만들도록 수정했습니다.

* 🚀 Java sdk 0.1.1 추가
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.

3 participants