Skip to content

Convention

seonyoung edited this page Jul 4, 2023 · 4 revisions

git flow ์ „๋žต

main, develop, feature, hotfix, release

  • ๊ฐœ๋ฐœ ์™„๋ฃŒ๋œ feature branch๋Š” develop์— ๋จธ์ง€ ํ›„ ์‚ญ์ œํ•ฉ๋‹ˆ๋‹ค.
  • feature ๋ธŒ๋žœ์น˜ ์ด๋ฆ„์€ feature/์ด์Šˆ๋ฒˆํ˜ธ-ํ‚ค์›Œ๋“œ (์˜ˆ : feature/8-UIComponent-button) ํ˜•์‹์œผ๋กœ ๋งŒ๋“ญ๋‹ˆ๋‹ค.

Commit Convention

  • [file] ํŒŒ์ผ ์ด๋™, ์‚ญ์ œ, ์ƒ์„ฑ
  • [feat] ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  • [fix] ๋ฒ„๊ทธ ์ˆ˜์ •
  • [setting] ํ”„๋กœ์ ํŠธ ์„ค์ • ๊ด€๋ จ (plist ์ˆ˜์ •์‹œ)
  • [refactor] ๋ฆฌํŒฉํ„ฐ๋ง
  • [test] ํ…Œ์ŠคํŠธ ์ฝ”๋“œ
  • [docs] ๋ฌธ์„œ ์ˆ˜์ • (readme, gitignore ๋“ฑ)
  • [chore] ํด๋” ์ˆ˜์ • ๋ฐ ๊ธฐํƒ€

์ปค๋ฐ‹ ๋ฉ”์„ธ์ง€ ๋ฐ ๋ณธ๋ฌธ ๋‚ด์šฉ์€ ํ•„์š” ์‹œ 2๋ฒˆ ๊ฐœํ–‰ ํ›„ ๊ฐ„๋žตํ•˜๊ฒŒ ์ ์Šต๋‹ˆ๋‹ค.

[feat] BaseView ์ถ”๊ฐ€ #8

- BaseView ๋‚ด configureUI, setupView ๊ตฌ์„ฑ

Coding Convention


Code Review ๊ทœ์น™

Pn ๊ทœ์น™

๋ฑ…ํฌ์ƒ๋Ÿฌ๋“œ์˜ Pn ๋ฃฐ ์„ ๋”ฐ๋ฆ…๋‹ˆ๋‹ค

P1: ๊ผญ ๋ฐ˜์˜ํ•ด์ฃผ์„ธ์š” (Request changes)
๋ฆฌ๋ทฐ ์š”์ฒญ์ž๋Š” p1 ํƒœ๊ทธ์— ๋Œ€ํ•ด ๋ฆฌ๋ทฐ์–ด์˜ ์š”์ฒญ์„ ๋ฐ˜์˜ํ•˜๊ฑฐ๋‚˜, ๋ฐ˜์˜ํ•  ์ˆ˜ ์—†๋Š” ํ•ฉ๋ฆฌ์ ์ธ ์˜๊ฒฌ์„ ํ†ตํ•ด ๋ฆฌ๋ทฐ์–ด๋ฅผ ์„ค๋“ํ•  ์ˆ˜ ์žˆ์–ด์•ผ ํ•ฉ๋‹ˆ๋‹ค.

P2: ์ ๊ทน์ ์œผ๋กœ ๊ณ ๋ คํ•ด์ฃผ์„ธ์š” (Request changes)
๋ฆฌ๋ทฐ ์š”์ฒญ์ž๋Š” P2์— ๋Œ€ํ•ด ์ˆ˜์šฉํ•˜๊ฑฐ๋‚˜ ๋งŒ์•ฝ ์ˆ˜์šฉํ•  ์ˆ˜ ์—†๋Š” ์ƒํ™ฉ์ด๋ผ๋ฉด ์ ํ•ฉํ•œ ์˜๊ฒฌ์„ ๋“ค์–ด ํ† ๋ก ํ•  ๊ฒƒ์„ ๊ถŒ์žฅํ•ฉ๋‹ˆ๋‹ค.

P3: ์›ฌ๋งŒํ•˜๋ฉด ๋ฐ˜์˜ํ•ด ์ฃผ์„ธ์š” (Comment)
๋ฆฌ๋ทฐ ์š”์ฒญ์ž๋Š” P3์— ๋Œ€ํ•ด ์ˆ˜์šฉํ•˜๊ฑฐ๋‚˜ ๋งŒ์•ฝ ์ˆ˜์šฉํ•  ์ˆ˜ ์—†๋Š” ์ƒํ™ฉ์ด๋ผ๋ฉด ๋ฐ˜์˜ํ•  ์ˆ˜ ์—†๋Š” ์ด์œ ๋ฅผ ๋“ค์–ด ์„ค๋ช…ํ•˜๊ฑฐ๋‚˜ ๋‹ค์Œ์— ๋ฐ˜์˜ํ•  ๊ณ„ํš์„ ๋ช…์‹œ์ ์œผ๋กœ(JIRA ํ‹ฐ์ผ“ ๋“ฑ์œผ๋กœ) ํ‘œํ˜„ํ•  ๊ฒƒ์„ ๊ถŒ์žฅํ•ฉ๋‹ˆ๋‹ค.

P4: ๋ฐ˜์˜ํ•ด๋„ ์ข‹๊ณ  ๋„˜์–ด๊ฐ€๋„ ์ข‹์Šต๋‹ˆ๋‹ค (Approve)
๋ฆฌ๋ทฐ ์š”์ฒญ์ž๋Š” P4์— ๋Œ€ํ•ด ์•„๋ฌด๋Ÿฐ ์˜๊ฒฌ์„ ๋‹ฌ์ง€ ์•Š๊ณ  ๋ฌด์‹œํ•ด๋„ ๊ดœ์ฐฎ์Šต๋‹ˆ๋‹ค.

P5: ๊ทธ๋ƒฅ ์‚ฌ์†Œํ•œ ์˜๊ฒฌ์ž…๋‹ˆ๋‹ค (Approve)
๋ฆฌ๋ทฐ ์š”์ฒญ์ž๋Š” P5์— ๋Œ€ํ•ด ์•„๋ฌด๋Ÿฐ ์˜๊ฒฌ์„ ๋‹ฌ์ง€ ์•Š๊ณ  ๋ฌด์‹œํ•ด๋„ ๊ดœ์ฐฎ์Šต๋‹ˆ๋‹ค.