Submit Search
Upload
플리토 코드리뷰 - Code Review in Flitto
•
13 likes
•
1,765 views
Yongjun Kim
Follow
'플리토 모바일팀의 코드리뷰 & IDE로 Intelligent 리팩토링' 사내세미나 발표자료입니다.
Read less
Read more
Software
Report
Share
Report
Share
1 of 58
Download now
Download to read offline
Recommended
임영기님 - 코드 리뷰 시스템 도입하기
임영기님 - 코드 리뷰 시스템 도입하기
OnGameServer
Code Review - DevOn2013
Code Review - DevOn2013
호정 이
NDC13: DVCS와 코드리뷰 그리고 자동화를 통한 쾌속 개발
NDC13: DVCS와 코드리뷰 그리고 자동화를 통한 쾌속 개발
Jinuk Kim
코드 리뷰 시스템 소개
코드 리뷰 시스템 소개
Young-Ho Cha
카카오스토리 웹팀의 코드리뷰 경험
카카오스토리 웹팀의 코드리뷰 경험
Ohgyun Ahn
C++ 코드 품질 관리 비법
C++ 코드 품질 관리 비법
선협 이
[AUG]개발자와 QA가 상생하는 테스트 프로세스
[AUG]개발자와 QA가 상생하는 테스트 프로세스
철민 신
IoT 개발자를 위한 Embedded C에서 TDD를 해보자
IoT 개발자를 위한 Embedded C에서 TDD를 해보자
Taeyeop Kim
Recommended
임영기님 - 코드 리뷰 시스템 도입하기
임영기님 - 코드 리뷰 시스템 도입하기
OnGameServer
Code Review - DevOn2013
Code Review - DevOn2013
호정 이
NDC13: DVCS와 코드리뷰 그리고 자동화를 통한 쾌속 개발
NDC13: DVCS와 코드리뷰 그리고 자동화를 통한 쾌속 개발
Jinuk Kim
코드 리뷰 시스템 소개
코드 리뷰 시스템 소개
Young-Ho Cha
카카오스토리 웹팀의 코드리뷰 경험
카카오스토리 웹팀의 코드리뷰 경험
Ohgyun Ahn
C++ 코드 품질 관리 비법
C++ 코드 품질 관리 비법
선협 이
[AUG]개발자와 QA가 상생하는 테스트 프로세스
[AUG]개발자와 QA가 상생하는 테스트 프로세스
철민 신
IoT 개발자를 위한 Embedded C에서 TDD를 해보자
IoT 개발자를 위한 Embedded C에서 TDD를 해보자
Taeyeop Kim
TDD: Test Driven Development 첫번째 이야기
TDD: Test Driven Development 첫번째 이야기
Ji Heon Kim
[NDC13] 70명이 커밋하는 라이브 개발하기 (해외 진출 라이브 프로젝트의 개발 관리) - 송창규
[NDC13] 70명이 커밋하는 라이브 개발하기 (해외 진출 라이브 프로젝트의 개발 관리) - 송창규
ChangKyu Song
파이썬 TDD 101
파이썬 TDD 101
정주 김
2018 01-code review
2018 01-code review
Myeongseok Baek
Tdd ver.2
Tdd ver.2
Henry Lee
Ui test 자동화하기 - Selenium + Jenkins
Ui test 자동화하기 - Selenium + Jenkins
Chang Hak Yeon
유지보수를 고려한 SW 개발
유지보수를 고려한 SW 개발
도형 임
Test driven development
Test driven development
Jinho Song
깨끗한 코드 (클린 코드, Clean Code)
깨끗한 코드 (클린 코드, Clean Code)
Jay Park
TDD
TDD
Henry Lee
TDD&Refactoring Day 02: TDD
TDD&Refactoring Day 02: TDD
Suwon Chae
테스트 케이스와 SW 품질
테스트 케이스와 SW 품질
도형 임
테스트 자동화와 TDD(테스트 주도 개발방법론)
테스트 자동화와 TDD(테스트 주도 개발방법론)
KH Park (박경훈)
프로젝트 Xxx에 적용하고 싶은 개발방법
프로젝트 Xxx에 적용하고 싶은 개발방법
도형 임
Java 그쪽 동네는
Java 그쪽 동네는
도형 임
E1_Deview nhn애자일개발 tdd_질문답
E1_Deview nhn애자일개발 tdd_질문답
NAVER D2
Test Driven Development (TDD) basic
Test Driven Development (TDD) basic
Curt Park
D2 캠퍼스 세미나 - 학생 개발자에서 신입 개발자로 한단계 업그레이드 하기
D2 캠퍼스 세미나 - 학생 개발자에서 신입 개발자로 한단계 업그레이드 하기
Soojin Ro
2019 11-code review
2019 11-code review
Myeongseok Baek
Node.js in Flitto
Node.js in Flitto
SeungWoo Lee
Where is CEO?
Where is CEO?
Yongjun Kim
On ly 2.0 project Review
On ly 2.0 project Review
soeun Lee
More Related Content
What's hot
TDD: Test Driven Development 첫번째 이야기
TDD: Test Driven Development 첫번째 이야기
Ji Heon Kim
[NDC13] 70명이 커밋하는 라이브 개발하기 (해외 진출 라이브 프로젝트의 개발 관리) - 송창규
[NDC13] 70명이 커밋하는 라이브 개발하기 (해외 진출 라이브 프로젝트의 개발 관리) - 송창규
ChangKyu Song
파이썬 TDD 101
파이썬 TDD 101
정주 김
2018 01-code review
2018 01-code review
Myeongseok Baek
Tdd ver.2
Tdd ver.2
Henry Lee
Ui test 자동화하기 - Selenium + Jenkins
Ui test 자동화하기 - Selenium + Jenkins
Chang Hak Yeon
유지보수를 고려한 SW 개발
유지보수를 고려한 SW 개발
도형 임
Test driven development
Test driven development
Jinho Song
깨끗한 코드 (클린 코드, Clean Code)
깨끗한 코드 (클린 코드, Clean Code)
Jay Park
TDD
TDD
Henry Lee
TDD&Refactoring Day 02: TDD
TDD&Refactoring Day 02: TDD
Suwon Chae
테스트 케이스와 SW 품질
테스트 케이스와 SW 품질
도형 임
테스트 자동화와 TDD(테스트 주도 개발방법론)
테스트 자동화와 TDD(테스트 주도 개발방법론)
KH Park (박경훈)
프로젝트 Xxx에 적용하고 싶은 개발방법
프로젝트 Xxx에 적용하고 싶은 개발방법
도형 임
Java 그쪽 동네는
Java 그쪽 동네는
도형 임
E1_Deview nhn애자일개발 tdd_질문답
E1_Deview nhn애자일개발 tdd_질문답
NAVER D2
Test Driven Development (TDD) basic
Test Driven Development (TDD) basic
Curt Park
D2 캠퍼스 세미나 - 학생 개발자에서 신입 개발자로 한단계 업그레이드 하기
D2 캠퍼스 세미나 - 학생 개발자에서 신입 개발자로 한단계 업그레이드 하기
Soojin Ro
2019 11-code review
2019 11-code review
Myeongseok Baek
What's hot
(19)
TDD: Test Driven Development 첫번째 이야기
TDD: Test Driven Development 첫번째 이야기
[NDC13] 70명이 커밋하는 라이브 개발하기 (해외 진출 라이브 프로젝트의 개발 관리) - 송창규
[NDC13] 70명이 커밋하는 라이브 개발하기 (해외 진출 라이브 프로젝트의 개발 관리) - 송창규
파이썬 TDD 101
파이썬 TDD 101
2018 01-code review
2018 01-code review
Tdd ver.2
Tdd ver.2
Ui test 자동화하기 - Selenium + Jenkins
Ui test 자동화하기 - Selenium + Jenkins
유지보수를 고려한 SW 개발
유지보수를 고려한 SW 개발
Test driven development
Test driven development
깨끗한 코드 (클린 코드, Clean Code)
깨끗한 코드 (클린 코드, Clean Code)
TDD
TDD
TDD&Refactoring Day 02: TDD
TDD&Refactoring Day 02: TDD
테스트 케이스와 SW 품질
테스트 케이스와 SW 품질
테스트 자동화와 TDD(테스트 주도 개발방법론)
테스트 자동화와 TDD(테스트 주도 개발방법론)
프로젝트 Xxx에 적용하고 싶은 개발방법
프로젝트 Xxx에 적용하고 싶은 개발방법
Java 그쪽 동네는
Java 그쪽 동네는
E1_Deview nhn애자일개발 tdd_질문답
E1_Deview nhn애자일개발 tdd_질문답
Test Driven Development (TDD) basic
Test Driven Development (TDD) basic
D2 캠퍼스 세미나 - 학생 개발자에서 신입 개발자로 한단계 업그레이드 하기
D2 캠퍼스 세미나 - 학생 개발자에서 신입 개발자로 한단계 업그레이드 하기
2019 11-code review
2019 11-code review
Viewers also liked
Node.js in Flitto
Node.js in Flitto
SeungWoo Lee
Where is CEO?
Where is CEO?
Yongjun Kim
On ly 2.0 project Review
On ly 2.0 project Review
soeun Lee
Everything You (N)ever Wanted to Know about Testing View Controllers
Everything You (N)ever Wanted to Know about Testing View Controllers
Brian Gesiak
エンジニア 李昇禹(イスンウ) 履歴書 (20160410)
エンジニア 李昇禹(イスンウ) 履歴書 (20160410)
SeungWoo Lee
Test driven developement
Test driven developement
Bhavik Panchal
2014 CANNES LIONS REVIEW (2014 칸느 국제광고제 수상작 리뷰)
2014 CANNES LIONS REVIEW (2014 칸느 국제광고제 수상작 리뷰)
Minyoung Jeong
[0820 석재호]headfirst디자인패턴
[0820 석재호]headfirst디자인패턴
Jaeho Seok
D2 OPEN SEMINAR - 개발자에 의한, 개발자를 위한 테스트
D2 OPEN SEMINAR - 개발자에 의한, 개발자를 위한 테스트
NAVER D2
모임 뒤에 사람있어요 - 9XD 1년 회고
모임 뒤에 사람있어요 - 9XD 1년 회고
Yurim Jin
GoF의 디자인 패턴
GoF의 디자인 패턴
Ashal aka JOKER
알고리즘과 자료구조
알고리즘과 자료구조
영기 김
차량용 소프트웨어 개발 시 소프트웨어 아키텍처 고려사항
차량용 소프트웨어 개발 시 소프트웨어 아키텍처 고려사항
신승환
깃헙으로 코드리뷰 하기
깃헙으로 코드리뷰 하기
Ohgyun Ahn
iOS human interface guidelines(HIG)
iOS human interface guidelines(HIG)
Sun Jin Choi
진짜기초 Node.js
진짜기초 Node.js
Woo Jin Kim
Helpful Review Recommendation (리뷰 추천시스템)
Helpful Review Recommendation (리뷰 추천시스템)
FAST CAMPUS
How to successfully grow a code review culture
How to successfully grow a code review culture
Nina Zakharenko
3 Things Every Sales Team Needs to Be Thinking About in 2017
3 Things Every Sales Team Needs to Be Thinking About in 2017
Drift
How to Become a Thought Leader in Your Niche
How to Become a Thought Leader in Your Niche
Leslie Samuel
Viewers also liked
(20)
Node.js in Flitto
Node.js in Flitto
Where is CEO?
Where is CEO?
On ly 2.0 project Review
On ly 2.0 project Review
Everything You (N)ever Wanted to Know about Testing View Controllers
Everything You (N)ever Wanted to Know about Testing View Controllers
エンジニア 李昇禹(イスンウ) 履歴書 (20160410)
エンジニア 李昇禹(イスンウ) 履歴書 (20160410)
Test driven developement
Test driven developement
2014 CANNES LIONS REVIEW (2014 칸느 국제광고제 수상작 리뷰)
2014 CANNES LIONS REVIEW (2014 칸느 국제광고제 수상작 리뷰)
[0820 석재호]headfirst디자인패턴
[0820 석재호]headfirst디자인패턴
D2 OPEN SEMINAR - 개발자에 의한, 개발자를 위한 테스트
D2 OPEN SEMINAR - 개발자에 의한, 개발자를 위한 테스트
모임 뒤에 사람있어요 - 9XD 1년 회고
모임 뒤에 사람있어요 - 9XD 1년 회고
GoF의 디자인 패턴
GoF의 디자인 패턴
알고리즘과 자료구조
알고리즘과 자료구조
차량용 소프트웨어 개발 시 소프트웨어 아키텍처 고려사항
차량용 소프트웨어 개발 시 소프트웨어 아키텍처 고려사항
깃헙으로 코드리뷰 하기
깃헙으로 코드리뷰 하기
iOS human interface guidelines(HIG)
iOS human interface guidelines(HIG)
진짜기초 Node.js
진짜기초 Node.js
Helpful Review Recommendation (리뷰 추천시스템)
Helpful Review Recommendation (리뷰 추천시스템)
How to successfully grow a code review culture
How to successfully grow a code review culture
3 Things Every Sales Team Needs to Be Thinking About in 2017
3 Things Every Sales Team Needs to Be Thinking About in 2017
How to Become a Thought Leader in Your Niche
How to Become a Thought Leader in Your Niche
Similar to 플리토 코드리뷰 - Code Review in Flitto
초보개발자의 TDD 체험기
초보개발자의 TDD 체험기
Sehun Kim
VSTS와 Azure를 이용한 팀 프로세스 관리
VSTS와 Azure를 이용한 팀 프로세스 관리
Gyuwon Yi
애자일 하라
애자일 하라
진수 허
200820 NAVER TECH CONCERT 15_Code Review is Horse(코드리뷰는 말이야)(feat.Latte)
200820 NAVER TECH CONCERT 15_Code Review is Horse(코드리뷰는 말이야)(feat.Latte)
NAVER Engineering
전통적인 개발과 테스트 주도 개발, 그리고 애자일
전통적인 개발과 테스트 주도 개발, 그리고 애자일
Tap ToRestart
2022 01-okky-코드리뷰
2022 01-okky-코드리뷰
Myeongseok Baek
더 나은 SW프로젝트를 위해
더 나은 SW프로젝트를 위해
지수 윤
테스트수행사례 W통합보안솔루션
테스트수행사례 W통합보안솔루션
SangIn Choung
오픈 스펙을 대상으로 한 테스트설계사례
오픈 스펙을 대상으로 한 테스트설계사례
SangIn Choung
(편집-테스트카페 발표자료) 1인 QA 수행사례로 발표한 자료 (W프로젝트 사례)
(편집-테스트카페 발표자료) 1인 QA 수행사례로 발표한 자료 (W프로젝트 사례)
SangIn Choung
엔지니어의 학습, 그리고 테스트 코드
엔지니어의 학습, 그리고 테스트 코드
Mijeong Park
[네이버오픈소스세미나] egjs-view360 개발기 - 김희재
[네이버오픈소스세미나] egjs-view360 개발기 - 김희재
NAVER D2
TDD로 Widget 개발하기
TDD로 Widget 개발하기
Bansook Nam
클린코드와 테스트코드
클린코드와 테스트코드
Herren
풀리퀘를 부탁해!
풀리퀘를 부탁해!
Mickey SJ Lee
Custom assert
Custom assert
Vincent Kwak
Software Inspection
Software Inspection
Samsung Electronics
공사꾼 개발부장 김종찬_페어코딩으로 테스팅 배우기_ausg_20170824
공사꾼 개발부장 김종찬_페어코딩으로 테스팅 배우기_ausg_20170824
AWSKRUG - AWS한국사용자모임
Learning Unit Testing with Pair Programming
Learning Unit Testing with Pair Programming
Jongchan Kim
리팩토링
리팩토링
Wonjun Hwang
Similar to 플리토 코드리뷰 - Code Review in Flitto
(20)
초보개발자의 TDD 체험기
초보개발자의 TDD 체험기
VSTS와 Azure를 이용한 팀 프로세스 관리
VSTS와 Azure를 이용한 팀 프로세스 관리
애자일 하라
애자일 하라
200820 NAVER TECH CONCERT 15_Code Review is Horse(코드리뷰는 말이야)(feat.Latte)
200820 NAVER TECH CONCERT 15_Code Review is Horse(코드리뷰는 말이야)(feat.Latte)
전통적인 개발과 테스트 주도 개발, 그리고 애자일
전통적인 개발과 테스트 주도 개발, 그리고 애자일
2022 01-okky-코드리뷰
2022 01-okky-코드리뷰
더 나은 SW프로젝트를 위해
더 나은 SW프로젝트를 위해
테스트수행사례 W통합보안솔루션
테스트수행사례 W통합보안솔루션
오픈 스펙을 대상으로 한 테스트설계사례
오픈 스펙을 대상으로 한 테스트설계사례
(편집-테스트카페 발표자료) 1인 QA 수행사례로 발표한 자료 (W프로젝트 사례)
(편집-테스트카페 발표자료) 1인 QA 수행사례로 발표한 자료 (W프로젝트 사례)
엔지니어의 학습, 그리고 테스트 코드
엔지니어의 학습, 그리고 테스트 코드
[네이버오픈소스세미나] egjs-view360 개발기 - 김희재
[네이버오픈소스세미나] egjs-view360 개발기 - 김희재
TDD로 Widget 개발하기
TDD로 Widget 개발하기
클린코드와 테스트코드
클린코드와 테스트코드
풀리퀘를 부탁해!
풀리퀘를 부탁해!
Custom assert
Custom assert
Software Inspection
Software Inspection
공사꾼 개발부장 김종찬_페어코딩으로 테스팅 배우기_ausg_20170824
공사꾼 개발부장 김종찬_페어코딩으로 테스팅 배우기_ausg_20170824
Learning Unit Testing with Pair Programming
Learning Unit Testing with Pair Programming
리팩토링
리팩토링
플리토 코드리뷰 - Code Review in Flitto
1.
CODE REVIEW
2.
눈치 보는 나,
오해하는 당신
3.
Yongjun Kim TeamA(pp) kevin@flitto.com
4.
켄트백 구현패턴 中
5.
HOW
6.
+
7.
feature/<new_feature_name> bugfix/<new_issue_name> bugfix/<JIRA issue number>
8.
masterdevelopbugfix branchesfeature branches tag/v2.4.1 tag/v2.4.0 tag/v2.5.0
9.
작업 마무리 되면
<develop> 으로 Pull Request
10.
Who is Reviewer?
11.
Android Developer 도
iOS 리뷰하자 iOS Developer 도 Android 리뷰하자
12.
작업 로그를 잘
남겨두자단, Dev.Log
13.
Code convention Commit Msg
format Dev.Log Not Only for Code Review, just for Team
14.
한번씩은 해보자. 주별로 Review
Master
15.
라인 단위 @리뷰
16.
만약 리뷰(댓글)가 있을
경우, 리뷰에 대한 피드백을 한다
17.
리뷰가 끝나면 최종 확인
메시지를 남긴다
18.
はやく한 작업이 아니라면 리뷰요청
진행작업
19.
그 날 받은
PR은 그 날 리뷰하자 Do Not 귀찮아
20.
Release Pull Request Merge <master> <develop> TAG QA
21.
22.
이 아닌….
23.
24.
오타 예외 처리 (exception,
null point) 라이브러리 추천 코드 제안 (로직 제안) Q&A 컨벤션(네이밍, 괄호, 들여쓰기, 주석)
25.
컨벤션(네이밍, 괄호, 들여쓰기,
주석) 발생빈도는 초기에는 높으나 초기 한달 정도는 좋음. PR 과정을 여러번 하다보면 저절로 없어짐, 컨벤션 확인을 자동화필요 컨벤션, 오타 찾기는 부가적 중요도는 낮아야 함
26.
!앗
27.
비난하기 위한 리뷰가
아닌 코드 품질을 위한 리뷰를 하자
28.
git stash git stash
list git stash pop 작업중 발생한 다른 이슈는 같은 브랜치에 포함하지 않음
29.
conflict으로 인한 auto- merging이
되지 않는 경우
30.
아직까지는 Push를 막지
않음 긴급 배포 git blame(비난) 받지 않을 만큼 테스트 후 바로 Push하거나 주위의 동료에게 코드리뷰를 요청
31.
만약 2명이상 작업을
하거나 작업이 큰 경우 ? developfeature branches feature/A feature/A1feature/A2 PR
32.
developfeature branches feature/b2b feature/b2b_dev Pull Pull Request Yesterday
33.
developfeature branches feature/b2b feature/b2b_dev Pull Request Pull Yesterday
34.
!아~
35.
장점
36.
코드를 읽게 되고
그를 통해 배움 로직, 예외 처리에 대한 리뷰와 논의로 프로젝트가 안정화 소수 인원의 온라인 리뷰로 인한 시간 절약 나보다 뛰어난 엔지니어의 리뷰로 인한 실력의 향상 코드리뷰를 통한 코드 품질 향상 변경이 용이한, 테스트 코드가 잘 동작하도록 미리 설계
37.
단점
38.
자주 하나의 PR에
많은 양의 코드 리뷰 가끔 iOS 로직을 모른다라는 핑계로 리뷰 집중력 저하 잦은 Pull Request로 리뷰 시간이 많이 소모됨 온라인 리뷰의 한계를 지님 (가끔 오프라인 리뷰도 필요) 시니어는 배움에 대한 한계
39.
어떤 자세가 필요한가?
40.
Reviewee commit 하기 전에
리팩토링이나 재확인 commit 단위를 잘 나누어 Pull Request 리뷰가 필요한지 안한지 잘 판단하는 것이 중요 리뷰에 대한 본인의 의도를 리뷰어에게 잘 설명함
41.
Reviewer 꼼꼼하게 되도록 라인단위의
리뷰 피드백이 마음에 들지 않는다면 추후에 직접 수정 서로간에 분쟁이 생길경우 리뷰이가 판단하도록 믿김 코드 품질보다 팀워크가 중요
42.
Flittoolcon
43.
commit 하기 전 IDE로
리팩토링 쉽게하기
44.
TDD Add TestCase Test Fail Write Code Run Test Refactor REPEAT
45.
TDD Pair Programming + = Good
46.
47.
Intelligent 한 Tool을
쓰니까 Intelligent 를 느껴봐야죠
48.
자주 쓰는 간단한
shortcuts
49.
어르신에게 친절하게 greet() 하지만
누군가는 move() 간단하게 믿고 따라하는 예제 hometown.js
50.
Github Refactoring Sample https://github.com/kimkevin/refactoring_sample
51.
시연
52.
Cmd + F6
: Rename Cmd + Opt + N : Inline Cmd + Opt + M : Extract Method Cmd + Opt + V : Extract Variable [추가]
53.
Cmd + F6
: Rename [추가]
54.
Cmd + Opt
+ N : Inline… (Function & Variable) [추가]
55.
Cmd + Opt
+ M : Extract Method [추가]
56.
Cmd + Opt
+ V : Extract Variable [추가]
57.
Q&A
58.
감사합니다 Thank you ありがとうございます Se lo
agradezco Terima kasih
Download now