Skip to content

Feat/init#4

Open
horobito wants to merge 10 commits intomainfrom
feat/init
Open

Feat/init#4
horobito wants to merge 10 commits intomainfrom
feat/init

Conversation

@horobito
Copy link
Contributor

@horobito horobito commented Apr 5, 2022

1. purpose in the pull request.

make modules and set module's gradle

2. What's New

  • new modules

3. what's changed

  • create new modules

4. Does the server run locally after the change?

  • none

5. Possible problems with this pr

  • none

6. Has it been verified through the test

  • none

@horobito
Copy link
Contributor Author

horobito commented Apr 5, 2022

#3

@aerain
Copy link
Member

aerain commented Apr 6, 2022

#3 pr comment test

import lombok.RequiredArgsConstructor;
import org.springframework.web.bind.annotation.RestController;

@RestController
Copy link
Member

Choose a reason for hiding this comment

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

reviews test

@@ -0,0 +1,4 @@
dependencies {
api project(":infrastructure")
Copy link
Member

Choose a reason for hiding this comment

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

도메인이 인프라 의존하면 안돼죠

build.gradle Outdated
implementation 'org.springframework.boot:spring-boot-autoconfigure:2.6.6'
testImplementation 'org.junit.jupiter:junit-jupiter-api:5.8.1'
testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine:5.8.1'
implementation 'org.projectlombok:lombok:1.18.22'
Copy link
Member

Choose a reason for hiding this comment

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

lombok은 컴파일타임에만 관여합니다

@@ -0,0 +1,3 @@
dependencies {
runtimeOnly 'mysql:mysql-connector-java'
Copy link
Member

Choose a reason for hiding this comment

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

infra에서는 application과 domain 의존하게 하면 됩니다.

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