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] .gitignore 파일 수정 및 gradle 파일 수정 #4

Merged
merged 4 commits into from
Nov 27, 2024

Conversation

uijin31
Copy link
Collaborator

@uijin31 uijin31 commented Nov 25, 2024

🔗 관련 이슈

Fixes #3

👩🏻‍💻 구현 내용

  • .gitignore 파일에서 불필요한 내용 삭제 및 주석 추가
  • 템플릿 주석에서 잘못된 부분 수정 (모든 키워드가 PR이 머지되면 닫히는데, 잘못 적어놨었음) (링크)
  • 공통 설정은 루트 프로젝트의 build.gradle로 이동 (플러그인 버전 등)

💬 PR 포인트 & 궁금한 점

  • 개인적으로 모든 프로젝트에서 공통된다고 생각한 부분을 루트 프로젝트의 build.gradle로 옮겼는데, 맞게 잘 옮긴건지 모르겠네요 😂

@uijin31 uijin31 added the ⚙ setting Extra attention is needed label Nov 25, 2024
@uijin31 uijin31 requested a review from f-lab-lyan November 25, 2024 07:01
@uijin31 uijin31 self-assigned this Nov 25, 2024
build.gradle Outdated
@@ -0,0 +1,22 @@
plugins {
id 'java'
Copy link
Collaborator

Choose a reason for hiding this comment

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

이렇게 하면, 루트 프로젝트가 java 프로젝트가 되지 않을까요? :-) 그런데 루트 프로젝트는 자바 프로젝트가 아닌데 말이죠...

Copy link

@uijin-j uijin-j Nov 26, 2024

Choose a reason for hiding this comment

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

아래 subprojects 블록에서 서브 프로젝트들의 자바 버전을 설정하려고 했는데, 그러려면 'java' 플러그인이 필요하더라구요😢 그런데 루트 프로젝트에 'java' 플러그인을 설정하면, Gradle이 자바 프로젝트로 인식하게 되니.. 아예 제거하고 자바 버전은 각 서브 프로젝트에서 관리하는게 좋겠네요! 7e49e6c

Copy link
Collaborator

@f-lab-lyan f-lab-lyan left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

@uijin31 uijin31 merged commit 2b0b42a into main Nov 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙ setting Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🎟️ gitignore 수정 및 이슈 템플릿 수정
3 participants