-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
/noti
|
src/main/resources/application.yml
Outdated
@@ -0,0 +1,3 @@ | |||
spring: | |||
profiles: | |||
active: local |
There was a problem hiding this comment.
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"
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사소한 부분이지만 신경써주어 감사합니다! 👍
/noti 한 가지 사소한 제안 남겼으나, 반영은 커찬 자유로 하면 될 것 같습니다. 이대로 approve 되어도 괜찮아요. > local.yml 파일의 gitignore 여부 저도 비토 의견과 동일하게 공통 local 환경 세팅이 유의하다고 생각해요. 이전 프로젝트에서 그렇게 사용하였고, 문제되는 부분이 크게 없었던 것 같아서요 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오우... 엄청 빠른 리뷰 반영 감사합니다 커찬 👏
🚩 연관 이슈
closed #8
🗣️ 리뷰 요구사항 (선택)
설명
추후 의논해야 할 사항
application-local.yml
은 .gitignore에 등록해서 개인마다 따로 가져가게 할까?