건강한 코드 리뷰 문화를 위한 첫걸음

1. 코드

2. 리뷰

3. 공감하기
~

4. 건강한

5. 코드

6. 리뷰

7. 문화

8. 만들기

9. ~

10. 이미

11. 잘

12. 알려진

13. 이야기들..
• Realm의 코드 리뷰, GitHub로 바로 적용하기
- https://realm.io/kr/news/codereview-howto
• 카카오스토리 코드 리뷰 도입 사례
- http://tech.kakao.com/2016/02/04/code-review
• 매끄러운 코드 리뷰를 돕는 10가지 방법
- http://www.bloter.net/archives/238819
• 티몬의 쿨(cool)한 개발자 되기 “코드 리뷰”에 대하여
- http://tmondev.blog.me/220701344824

14. 코드

15. 리뷰를

16. 잘

17. 하는

18. 방법,

20. 좋은

21. 문화로

22. 정착시킨

23. 경험은

25.
이미

26. 많이

27. 공유되어

28. 있다.

29. 알면서도

30. 시작하기

31. 쉽지

32. 않고,

34.
성공하기는

35. 더욱

36. 어렵다.

37. 가장

38. 중요한

39. 것은

40. 코드

41. 리뷰에

42. 대한

44. 구성원들

45. 간의

46. “공감대”를

47. 형성하는

48. 것

49. 코드

50. 리뷰에

51. 대해서

52. 구성원들이

54.
어떻게

55. 생각하고

56. 있는지

58.
서로

59. 알아가는

60. 것에서

62.
시작해보자

63. 그러면

64. 질문

65.
(a.k.a.

66. 말해

67. Yes

68. Or

69. NO)

70. 코드

71. 리뷰,

72. 과연

73. 필요한가?

74. NO라고

75. 말하면..

77. 이기적이고

78. 독단적으로

79. 비춰질까

80. 두렵다.

81. 일단은..

82. Yes!

83. 그럼

84. 왜

85. 필요할까요?

86. 내

87. 코드를

88. 남이

89. 보기

90. 때문에

92.
발코딩하면

93. 부끄럽다.

94. 자연스럽게

95. 내가

96. 짠

97. 코드를

98.
가장

99. 먼저

100. 리뷰하게

101. 됨

102. 문학계

103. 전문

104. 용어로

105. “퇴고”

106. 리뷰

107. 받을

108. 코드라고

109. 생각하면

111.
더욱

112. 신경

113. 써서

114. 코딩하게

115. 된다.

116.
(가독성,

117. 깔끔한

118. 로직)

119. 구성원들의

120. 코딩

121. 역량

122. 상향

123. 평준화

124.
(임작가님의

125. L.E.S.S.??)

126. 깨진

127. 유리창의

128. 법칙

129. • 내

130. 코드에

131. 버그가

132. 있는

133. 건

134. 아닐까?
(버그/장애

135. 조기

136. 발견)

137.
• 내가

138. 코드를

139. 과연

140. 잘

141. 짠걸까?
(가독성

142. /

143. 구조

144. /

145. 로직)

146.
• 저는

147. 이런

148. 작업을

149. 했습니다.
(히스토리,

150. 정보

151. 공유)
기타

152. 등등

153. 필요한

154. 건

155. 알겠는데

156. 잘

157. 안된다..

158. 오비삼척

159. Gwichanism,

160. 귀찮ist,

161. 게으르만족

162. 빨리

163. 배포

164. 나가야

165. 되는데,

166.
코드

167. 리뷰

168. 기다리고

169. 고칠

170. 시간이

171. 없다.

172.
(극현실주의)

173. 변경된

174. 코드가

175. 너무

176. 많아서

177.
볼

178. 엄두가

179. 안난다.

180. 사실

181. 봐도

182. 잘

183. 모르겠다ㅡ.,ㅡ

184.
익숙하지

185. 않은

186. 언어/프레임워크,

187.
잘

188. 모르는

189. 도메인

190. 영역에서

192.
특히...

193. 용기

194. 내서

195. PR

196. 날렸는데..

197.
돌아오는

198. 것은

199. 무관심ㅠㅠ

200. 나는

201. 주니어인데..

202. 실력이

203. 부족한데..

204.
내

205. 어찌

206. 감히..

207.
(코드

208. 리뷰에

209. 대한

210. 오해!!!!)

211. 여기서

212. 잠깐!

213.
~코드

214. 리뷰에

215. 대한

216. 오해~

217. 잘하는

218. 사람한테

219. 검사

220. 받는거다?

221. NEVER!

222. 커뮤니케이션

223. 활동의

224. 일부

225. 일
뿐

226. 전부가

227. 아니다!

228. 리뷰

229. 받은

230. 내용은

231. 무조건

232. 고친다?

233. NOPE!

234. 개개인의

235. 스타일의

236. 차이,

237. 배

238.
보다

239. 배꼽이

240. 더

241. 커질

242. 수

243. 있다

244. 시간이

245. 남아

246. 돌아서

247. 리뷰한다?

248. 괜히

249. 지적

250. 받는

251. 것

252. 같고,

253.
자존감이

254. 떨어진다???

255. 공감대

256. 부족

257. +

258. 리뷰어의

259. 잘못된

260. 리뷰

261. 어려움을

262. 겪었던

263. 경험..

264. 이슈 처리를 위해 달랑 한
줄만 고쳤는데, 주변 코드 까
지 묶어서 언급하면 괜히 억
울해진다.
‘내가 짠거 아닌데...ㅠㅠ’

265. ‘왜그렇게했어요’라
고 코멘트가 달리면, ‘내
가 뭘 잘못했나?’라는
생각에 의기소침해진다.

266. PR 설명이 너무 부족(불
친절)해서, 이해도 잘 안
되고 코드 리뷰에 시간이
너무 오래 걸려서 지친다.

267. 실제로 작업된 코드량은 적
은데 리포맷팅, 이름/패키지
변경등으로 변경이력이 너
무 많아서 리뷰하는데 불필
요한 시간을 낭비

268. 시간을 투자해서 열심
히 코멘트 달았는데 무
반응 merge or close 되
버리면 허탈함..

269. PR 안날리고 로컬에서 머
지하고 푸시하는 사람들은
편하게 잘 먹고 잘 사는
데, 나는 왜 고통받는가..

270. 코드

271. 리뷰에

272. 대한

273. 잘못된

274. 인식,

275. 공감대
의

276. 부재,

277. 배려심

278. 부족에서

279. 오는

280. 어려움

281.
(코드

282. 리뷰의

283. 단점이

284. 아님)

285. 좋았던

286. 경험

287. ^^

288. 버그

289. 조기

290. 발견으로

291. 장애를

292. 피함
(넘나

293. 친숙한

294. 케이스)

295. OOP와

296. clean

297. code에

298. 대한

299. 역량이

300.
매우

301. 향상됨

302. 초기에는

303. 코멘트들이

304. 많이

305. 달렸지만,

307.
시간이

308. 지날수록

309. 따봉(:+1)을

310. 많이

311. 받으면서

312.
기분이

313. 좋아짐

314. *^^*

315. 리뷰

316. 받을

317. 커밋과

318. 기타

319. (리포맷팅/이름,패키지

320. 변경

321.
등..)

322. 작업을

323. 별도의

324. 커밋으로

325. 분리하고,

326. PR

327. Description
에

328. 언급해주니

329. 코드

330. 리뷰

331. 시간이

332. 매우

333. 빨랐음

334. 보다

335. 나은

336. 코드

337. 스타일

338. 제안

340.
(그러나

341. 취향의

342. 차이일

343. 수도

344. 있음)

345. 작업/리뷰에

346. 직접

347. 참여하지는

348. 않았
어도

349. watching

350. 만으로도

351. 꿀팁

352. 줍줍

353. 리뷰이(요청자)의

354. 마음가짐

355. 코드

356. 리뷰가

357. 평가/지적

358. 받는

359. 행위라고

360.
생각하지

361. 말자.

362. 리뷰어

363. 입장에서

364.
본인이

365. 가장

366. 먼저

367. 리뷰하자.

368. 리뷰어가

369. 쉽고

370. 빠르게

371. 리뷰

372. 할

373. 수

374. 있도록

375.
PR

376. Description과

377. 커밋

378. 로그를

379. 잘

380. 활용
하자

381. 모든

382. 리뷰를

383. 다

384. 적용할

385. 필요는

386. 없다.

387.
부담가지지

388. 말자.

389. 그렇다고

390. 해서

391. 리뷰어의

392. 소중한

393. 코멘트
를

394. 놓치지

395. 말자.

396. 정답은

397. 없다.

398.
(버그가

399. 아니라면)제안/의견/팁

400. 정도로

401.
생각하자!

402. 리뷰어의

403. 마음가짐

404. 지적하거나

405. 평가하는

406. 늬앙스의

407. 코멘
트는

408. 절대

409. 하지

410. 말자!

411. “그건

412. 그렇게

413. 하면

414. 안돼요”

416.
보다는

418.
“이렇게

419. 해보면

420. 어떨까요”

421. 질문도

422. 자유롭게

423. 하자!

424. 바쁠

425. 때는

426. 중요

427. 포인트만이라도..!

428. 적극적으로

429. 따봉(:+1)을

430. 날리자

431. 그래도

432. 조금은

433. 엄격해도

434. 되는

435. 것들
• 컨벤션을

436. 지키지

437. 않는

438. 코드들

439.
• 가독성이

440. 매우

441. 떨어지는

442. 코드들

443.
• 기존의

444. 구조를

445. 망가트리는

446. 코드들

447.
• 안전망(테스트

448. 코드)

449. 없이

450. 크게

451. 변경되는

452. 코드들

453. 마지막

454. 강제한다고

455. 문화가

456. 되지

457. 않는다.

458. 공감

459. 없는

460. 리뷰는

461. 오해를

462. 유발한다

463. 오해가

464. 생기면

465. 감정

466. 낭비,

467. 스트레스..

468. 공감대가

469. 있어야

470.
건강한

471. 문화로

472. 발전할

473. 수

474. 있다.

475. 너무

476. 부담

477. 가지지

478. 말고

479. 공감대가

480. 형성

481. 되었다면..

483.
마음

484. 껏

485. 리뷰

486. 해봅시다.

487. ㄲㅡㅌ

488. 부록

489. 워크플로우에

490. 코드리뷰

491. 녹이기

492. git

493. flow

495. github

496. flow

497. http://nvie.com/posts/a-successful-git-branching-model/

498. https://guides.github.com/introduction/flow/

499. • start

500. feature

501. (git-flow)

502.
• commit

503. -

504. commit

505.
• test

506.
• github

507. develop

508. ~

509. feature

510. Pull

511. Request

512.
• REVIEW

513. :

514. comment

515. -

516. comment

517. -

518. commit

519. -

520. comment

521.
• github

522. PR

523. merge(with

524. Squash)

526. Delete

527. branch

528. -

529. finished

530. feature

531.
• start

532. release

533. (git-flow)

534.
• master

535. ~

536. release

537. Pull

538. Request

539.
• REVIEW

541.
• finish

542. release

543. (git-flow)

544. -

545. closed

546. PR

547. make

548. best

549. practices