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

Issue 37 #177

Merged
merged 17 commits into from
Oct 25, 2018
Merged

Issue 37 #177

merged 17 commits into from
Oct 25, 2018

Conversation

fuzzythecat
Copy link
Member

리뷰 미리 감사드립니다!

@parkjh688
Copy link
Collaborator

뭔가 적어드리고 싶었는데, 꼼꼼하게 적어주셔서 적을 것이 없네요😂

고생하셨습니다!

Copy link
Member

@MijeongJeon MijeongJeon left a comment

Choose a reason for hiding this comment

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

@fuzzythecat 연준님! 정말 깜끔하게 작업해주셨네요. 멋집니다.
코드 부분에 있는 주석은 해석 의도적(?)으로 안하신건지 ㅎㅎ 그부분 확인되면 바로 머지하겠습니다.

@fuzzythecat
Copy link
Member Author

@MijeongJeon 의도적은 아니고 깜빡했습니다..ㅠㅠ 수정했습니다!

@MijeongJeon MijeongJeon merged commit fe3d096 into master Oct 25, 2018
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.

None yet

3 participants