Skip to content

park7777/이슈이름 : Set up mybatis.#11

Open
beanbroker wants to merge 1 commit intomasterfrom
park7777
Open

park7777/이슈이름 : Set up mybatis.#11
beanbroker wants to merge 1 commit intomasterfrom
park7777

Conversation

@beanbroker
Copy link
Contributor

park7777/Set up mybatis.

#10

@jhmin1993 jhmin1993 requested review from jhmin1993 and removed request for jhmin1993 April 13, 2017 07:59
@ttobii
Copy link
Contributor

ttobii commented Apr 25, 2017

conflict 를 해결해 보세요 Github 상의 에디터로 해결할만한 간단한 문제입니다.


<dependency>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-configuration-processor</artifactId>
Copy link
Contributor

Choose a reason for hiding this comment

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

pull request 내용에 해당 라이브러리에 대한 내용과 필요성을 추가해주세요
해당 내용은 @it2seiyon 님의 리뷰가 필요합니다. pom 파일 담당자이니까요 !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ConfigurationProperties 을 사용할 때는 spring-boot-configuration-processor를 클래스패스에 설정하는 것을 권장한다는 정보를 발견하였습니다.

정보는 아래와 같습니다.
STS 에서 간단하게 스프링부트 애플리케이션 생성하기
STS 에서 부트 애플리케이션을 실행하고 디버깅하기
STS Properties editor를 이용해서 설정프로퍼타이즈 편집하기
@ConfigurationProperties를 사용하는 코드에서 설정프로퍼타이즈 편집하기

timerName: exampleTimer123
datasource:
driver-class-name: com.mysql.jdbc.Driver
---
Copy link
Contributor

Choose a reason for hiding this comment

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

이전에 실습했던 양식으로 공통 설정은 reousrce/application.yml 로 에 넣어주시고
환경에 따라 달라지는 database 설정만 resource-local/application-profile.yml 혹은 resource-{name}/application-profile.yml
에 넣어주세요

Copy link
Contributor

@ttobii ttobii left a comment

Choose a reason for hiding this comment

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

아래 코멘트 단 부분을 해결해 주세요
그리고 conflict 난 부분도 해결해야 합니다.

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.

2 participants