-
Notifications
You must be signed in to change notification settings - Fork 0
Convention
gijin noh edited this page Jan 10, 2021
·
5 revisions
- ํ์ผ๋ช ์ ๊ธฐ๋ณธ์ ์ผ๋ก PascalCase๋ก ์์ฑํ๋ค.
- ์ปดํฌ๋ํธ ํ์ผ๋ช ๊ณผ style ํ์ผ๋ช ์ ๊ฐ๋ค.
์์: MainPage.js, MainPage.module.scss
- ๋ณ์๋ช ์ ๊ธฐ๋ณธ์ ์ผ๋ก camelCase๋ฅผ ์ฌ์ฉํ๋ค.
- ๊ฐ๋ฅํ ๋ช ์ฌ๋ฅผ ์ฌ์ฉํ๋ค.
- ๋ณต์์ ๊ฒฝ์ฐ ๋ณต์ํ ๋ช ์ฌ๋ฅผ ์ฌ์ฉํ๋ค.
- boolean ํ์ ๊ฐ๋ ๋ณ์๋ ์์ is๋ฅผ ๋ถ์ธ๋ค.
- ํจ์๋ช ์ ๊ธฐ๋ณธ์ ์ผ๋ก camelCase๋ฅผ ์ฌ์ฉํ๋ค.
- ๊ฐ๋ฅํ ๋์ฌ๋ฅผ ์ฌ์ฉํ๋ค.
- ํจ์๋ช ์ ์ถ์ฝํ์ง ์๋๋ค.
- booleanํ์ returnํ๋ ํจ์๋ ์์ check๋ฅผ ๋ถ์ธ๋ค.
- ์คํ์ผ ์ฌ์ด์ ๊ตฌ๋ถ์๋ -๋ฅผ ์ฌ์ฉํ๋ค.
- Issue ๋ด์ฉ์ Todo-List ํ์์ผ๋ก ์์ฑํ๋ค.
-
์ปค๋ฐ ์ ์๋ ๊ท์น์ ์ฐธ๊ณ ํด ๋ฉ์ธ์ง๋ฅผ ์์ฑํ์, ์์ด๋ก ์์ฑํ ํ์๋ ์๋ค.
-
issue-number๋ Github์ issue๋ฅผ ๋ฐ๊ธํ๊ณ , ์์ฑ๋ ๋ฒํธ๋ฅผ ์ฌ์ฉํ๋ค.
[#{issue-number}] feat - (feature)
[#{issue-number}] fix - (bug fix)
[#{issue-number}] refactor - (split package)
[#{issue-number}] docs - (documentation)
[#{issue-number}] style - (formatting, missing semi colons, โฆ)
[#{issue-number}] test - (when adding missing tests)
master
dev
feature/{issue-number}-{desc..}
- branch๋ camelCase๋ก ์์ฑํ๋ค.
- Pull Request๋ Merge ์ Reviewer์ approve๊ฐ ํ์ํ๋ค.
- feature -> dev -> master
- VS Code์ eslint์ prettier extention์ ํ์ฉํ๋ค.
- ๋ฒ๊ทธ๊ฐ ์์ผ๋ฉด ์ปค๋ฐํ ์ ์๋ค.
- ํ ๋ฉ์๋์ ์ค์ง ํ ๋จ๊ณ์ ๋ค์ฌ์ฐ๊ธฐ๋ฅผ ํ๊ธฐ์ํด ๋ ธ๋ ฅํ๋ค.
- else ์์ฝ์ด๋ ๊ธ์งํ๋ค.
- ์์๋ ๋ณ๋์ ํ์ผ๋ก ๋ถ๋ฆฌํด์ ์์ฑํ๋ค.