Repository navigation
π’ ν νλ‘μ νΈ λ¨Έμ§ κ·μΉ(Merge Rule) νμ€ μ μ© μλ΄ #3
frydaynar
announced in
Announcements
Replies: 1 comment
|
νμΈνμ΅λλ€ |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
β Merge Rule (λ¨Έμ§ κ·μΉ)
1. λμ λ°°κ²½ λ° λͺ©μ
μμ μ±μ΄ νμΈλ μ½λλ§
mainλΈλμΉμ λ°μνμ¬ μλΉμ€ μ λ’°λλ₯Ό ν보ν©λλ€.κ³΅ν΅ νλ«νΌμ μ΄μ μμΉμ ν νλ‘μ νΈμ κ·Έλλ‘ μ΄μνμ¬ νμ ν¨μ¨μ κ·Ήλνν©λλ€.
2. ν΅μ¬ λ¨Έμ§ κ·μΉ (Merge Rules)
μ€λ¬΄ν κ°λ° μ΄μμ μν΄ λ€μ κ·μΉμ μ격ν μ€μν©λλ€.
mainμ§μ Push κΈμ§λͺ¨λ λ³κ²½ μ¬νμ λ°λμ Pull Request(PR)λ₯Ό ν΅ν΄μλ§ λ°μλ μ μμ΅λλ€.
μ΅μ 1μΈ μ΄μμ νμμ μΉμΈ(Approve) μ΄ μμ΄μΌ λ³ν©μ΄ κ°λ₯ν©λλ€.
Merge μ λ°μνλ μ½λ μΆ©λμ PR μμ±μ(μμ λ΄λΉμ) κ° μ§μ ν΄κ²°νλ κ²μ μμΉμΌλ‘ ν©λλ€.
Mergeκ° μλ£λ μμ λΈλμΉ(
feature/*,update/*λ±)λ μ μ₯μ κ΄λ¦¬λ₯Ό μν΄ μ¦μ μμ ν©λλ€.3. PR & Merge νλ‘μΈμ€
μμ μλ£ ν λ³ν©κΉμ§μ νμ€ μ μ°¨λ λ€μκ³Ό κ°μ΅λλ€.
κΈ°λ₯ λ¨μ μμ μλ£ ν, UI νλ©΄ λμ κ²μ¦(Frontend) λλ API μ μ λμ λ° ν΅μ¬ λ‘μ§ κ²μ¦(Backend/AI)μ λ§μΉ λ€ PRμ μμ±ν©λλ€. (λ―Έμμ± κΈ°λ₯ PR μ§μ)
PR μμ± μ μ§μ λ PR Templateμ μ¬μ©νλ©°, μ£Όμ λ³κ²½ μ¬ν λ° λ¦¬λ·° ν¬μΈνΈλ₯Ό μμΈν κΈ°λ‘ν©λλ€.
Closes #Issueλ²νΈνμμ μ¬μ©νμ¬ ν΄λΉ μμ κ³Ό μ°κ΄λ μ΄μκ° μλμΌλ‘ λ«νλλ‘ μ€μ ν©λλ€.리뷰μ΄μ νΌλλ°±μ λ°μνμ¬ μ½λλ₯Ό μμ ν ν μ¬μμ²ν©λλ€.
λͺ¨λ μΉμΈμ΄ μλ£λλ©΄ μ΅μ’ νμΈ ν
devλ°mainλΈλμΉμ ν΅ν©ν©λλ€.4. μ΅μ’ κΆν λ° μ΄μ
ν νλ‘μ νΈμ μμ μ μΈ ν΅ν©μ μν΄ νμ₯(@HorieZA) μ΄ μ΅μ’ νμΈ ν
devλ°mainλΈλμΉ λ³ν©μ λ΄λΉνλ©°, λΆμ¬ μ λ³ΈμΈ μΈ νμ(@mpsddd-commits, @frydaynar) μ΄ λνν©λλ€5. PR ν νλ¦Ώ (PR Template)
π κ΄λ ¨ κ·Όκ±° λ° λ¬Έμ
μ°λ¦¬ νμ "κ²ν λμ§ μμ λ³κ²½μ ν μμ°μ΄ λ μ μλ€" λ μ΄μ μμΉμ λ°νμΌλ‘ μμ§μ λλ€.
νΉν μ ννΈ κ°μ κΈ΄λ°ν νλ©΄ λ° API μ°λμ΄ νμμ μΈ M3 λ¨κ³μΈ λ§νΌ, μλ‘μ μ½λλ₯Ό κΌΌκΌΌν 리뷰νμ¬ μμ±λ λμ μμ€ν μ λ§λ€μ΄ μ£ΌμκΈ° λ°λλλ€.
All reactions