Skip to content

Conversation

@onys-programmer
Copy link

No description provided.

상품 서비스의 public 메서드들에 javadoc형식으로 주석을 작성했습니다.
상품 컨트롤러의 public 메소드들에 javadoc 형식으로 주석을 작성했습니다.
상품 도메인의 public 메서드에 javadoc 형식의 주석을 작성했습니다.
Copy link
Collaborator

@johngrib johngrib left a comment

Choose a reason for hiding this comment

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

6a603e4 커밋 메시지 타이틀에는 마침표를 찍지 않습니다.

상품 서비스를 문서화하라. -> 상품 서비스를 문서화하라
image

Copy link
Collaborator

@johngrib johngrib left a comment

Choose a reason for hiding this comment

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

오래간만입니다. 이번주도 열심히 해 봅시다!

@onys-programmer
Copy link
Author

오래간만입니다. 이번주도 열심히 해 봅시다!

이번주는 레고나 프라모델을 내 마음대로 쪼물딱거리는 느낌으로
프로젝트에 제 숨결을 불어넣어보려고 합니다. ㅎㅎ
응원 감사합니다.

명확하고 간단한 주석이 되도록 수정하고 필요없는 공백을 삭제했습니다.
회원 서비스의 public 메소드들의 역할을 javadoc으로 작성하였습니다
@johngrib johngrib mentioned this pull request Mar 18, 2021
Copy link
Collaborator

@johngrib johngrib left a comment

Choose a reason for hiding this comment

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

주석도 좋지만 이제 Docker 작업을 하셔야 하지 않을까요?!

@onys-programmer
Copy link
Author

주석도 좋지만 이제 Docker 작업을 하셔야 하지 않을까요?!

도전해보겠습니다!

application.yaml과 build.gradle에 mariadb를 적용했습니다
profiles: mariadb
url: jdbc:mariadb://mariadb:3306/test
username: root
password: root1234
Copy link
Collaborator

Choose a reason for hiding this comment

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

파일 마지막에 공백 라인을 추가해 주세요.

Copy link
Collaborator

@johngrib johngrib left a comment

Choose a reason for hiding this comment

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

주간 회고를 작성해 공유해주세요.

@johngrib johngrib merged commit 3da09f9 into CodeSoom:onys-programmer Mar 22, 2021
Copy link
Collaborator

@johngrib johngrib left a comment

Choose a reason for hiding this comment

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

한 주간 고생 많으셨습니다.

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