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] 초기 프로젝트 세팅 #9

Merged
merged 2 commits into from
Dec 11, 2024
Merged

[FEAT] 초기 프로젝트 세팅 #9

merged 2 commits into from
Dec 11, 2024

Conversation

leegwichan
Copy link
Contributor

@leegwichan leegwichan commented Dec 10, 2024

🚩 연관 이슈

closed #8

🗣️ 리뷰 요구사항 (선택)

설명

  • .gitignore는 주로 폴더/패키지를 등록하는데 빈 파일로 사용
    • 추후 해당 폴더/패키지에 파일을 추가할 때 삭제해주어야 한다
  • Swagger 관련 의존성은 콜리가 추가해 주세요~

추후 의논해야 할 사항

  • application.yml을 사용하는데, 환경을 어떻게 운용할까?
    • ex. application-local.yml은 .gitignore에 등록해서 개인마다 따로 가져가게 할까?

@unifolio0 unifolio0 added the feat 기능 추가 label Dec 10, 2024
@unifolio0
Copy link
Contributor

/noti
approve합니다~

pplication.yml을 사용하는데, 환경을 어떻게 운용할까?
ex. application-local.yml은 .gitignore에 등록해서 개인마다 따로 가져가게 할까?

application-local.yml도 왠만해선 공통으로 사용했던 것 같아요!

coli-geonwoo
coli-geonwoo approved these changes Dec 11, 2024
@@ -0,0 +1,3 @@
spring:
profiles:
active: local
Copy link
Contributor

Choose a reason for hiding this comment

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

[사소한 제안]

아마 이렇게 되면 정적으로 local이 active 될 것 같아서, default 설정을 통해 로컬 환경에서 개발 시 local 환경을 기본 설정으로 한다 라는 미묘한 의미를 주어보는 것은 어떤가요?

spring:
  profiles:
    default: "local"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

반영 완료했습니다~

@@ -0,0 +1 @@
rootProject.name = 'debatetimer'
Copy link
Contributor

Choose a reason for hiding this comment

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

사소한 부분이지만 신경써주어 감사합니다! 👍

@coli-geonwoo
Copy link
Contributor

/noti
커찬! 프로젝트 세팅을 위해 애써주어서 고마워요 🙇

한 가지 사소한 제안 남겼으나, 반영은 커찬 자유로 하면 될 것 같습니다. 이대로 approve 되어도 괜찮아요.

> local.yml 파일의 gitignore 여부

저도 비토 의견과 동일하게 공통 local 환경 세팅이 유의하다고 생각해요. 이전 프로젝트에서 그렇게 사용하였고, 문제되는 부분이 크게 없었던 것 같아서요 😄

Copy link
Contributor

Choose a reason for hiding this comment

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

오우... 엄청 빠른 리뷰 반영 감사합니다 커찬 👏

@leegwichan leegwichan merged commit 3537f6e into develop Dec 11, 2024
@leegwichan leegwichan deleted the feat/#8 branch December 11, 2024 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 프로젝트 기본 설정
3 participants