-
Notifications
You must be signed in to change notification settings - Fork 2
Convention
Yunseon Choi edited this page May 6, 2021
·
4 revisions
- LAYOUTS : WHAT_WHERE.XML / e.g. activity_main.xml
- STRINGS : WHERE_DESCRIPTION / e.g. main_intro, all_done
- DRAWABLES : WHERE_DESCRIPTION_SIZE / e.g. main_background
- IDS : WHAT_WHERE_DESCRIPTION / e.g. linearylayout_main_fragmentcontainer
- DIMENSIONS : WHAT_WHERE_DESCRIPTION_SIZE / e.g. keyline_all_text
- Fagment๋ PascalCase๋ก ์์ฑํ๋ค. / e.g BookFragment
- ๋ณ์๋ช ์ ๊ธฐ๋ณธ์ ์ผ๋ก 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)
masterdevfeature/{issue-number}-{desc..}- branch๋ camelCase๋ก ์์ฑํ๋ค.
- Pull Request๋ Merge ์ Reviewer์ approve๊ฐ ํ์ํ๋ค.
- feature -> dev -> main
- Android Lint๋ฅผ ํ์ฉํ๋ค.
- ๋ฒ๊ทธ๊ฐ ์์ผ๋ฉด ์ปค๋ฐํ ์ ์๋ค.
- import๋ฅผ ์ ๊ทํํ๋ค.
import foo.* (x)
import foo.bar (o)
- ๋ณ์ ๋ฒ์๋ ๋ช ํํ๊ฒ ์ ํํ๋ค.
- ์์ธ๋ฅผ ๋ฌด์ํ ๊ฒฝ์ฐ ์ฃผ์์ผ๋ก ์ด์ ๋ฅผ ๋จ๊ธด๋ค.
- ๋ค์ฌ์ฐ๊ธฐ๋ 4์นธ์ผ๋ก ํ๋ค.