แต่ละท่านตอน commit git มี standard การเขียนอย่างไรกันบ้าง

แตก branch แล้วทำแต่ละ

bluezip Wed, 24/07/2019 - 15:45

แตก branch แล้วทำแต่ละ features ให้เสร็จและก็ merge เข้า production มั่ง

ผมเพิ่งย้ายมาใช้ https://www

armariya Wed, 24/07/2019 - 19:09

ผมเพิ่งย้ายมาใช้ https://www.conventionalcommits.org/en/v1.0.0-beta.4/ อันนี้ ผมว่าก็โอเคนะครับ อ่านง่าย

“fixed fn aaa”

bellpocket Wed, 24/07/2019 - 20:13

“fixed fn aaa”

ของผมไม่สนใจ commit

ZiiT Thu, 25/07/2019 - 11:06

ของผมไม่สนใจ commit description เพราะถือเป็น work in progress แต่จะไปเน้นที่ Pull Request (PR) แทน โดยมีรายละเอียดประมาณนี้

PR Title สื่อความหมาย
PR ต้อง Link กับงาน (Work Item/Bug)
PR มี Code reviewer approve อย่างน้อย 2 คน, approve status จะถูก reset ถ้ามี code ใหม่ push ขึ้นไป
PR Build CI ผ่าน
Merge PR แบบ squash merge ทำให้ใน Master branch จะเห็นเป็น commit เดี๋ยวโดยมี PR Number/Title เป็น Commit description เพื่อไม่ให้ History ของ Master เปื้อนไปด้วย commit description พวก "fix bug", "fix typo" ที่ไม่มีประโยชน์เวลากลับมาย้อนดู

+1 น่าสนใจ

bellpocket Tue, 30/07/2019 - 09:14

+1 น่าสนใจ

ผมว่า commit msg

ripzery Thu, 01/08/2019 - 22:08

ผมว่า commit msg มีประโยชน์กับ reviewer อยู่ดีนะ

ใช่ครับ แต่มีประโยชน์แค่ใน

ZiiT Fri, 02/08/2019 - 11:26

ใช่ครับ แต่มีประโยชน์แค่ใน Pull Request นั้นๆ พอ Merge เข้าไปใน Master branch แล้ว สำหรับผมมันไม่มีประโยชน์แล้ว เพราะหลายๆครั้งจะเป็น Refactor xx, fix comment yyy

แต่ส่วนมากระบบมันก็ดึง commit

whitebigbird Fri, 02/08/2019 - 18:40

แต่ส่วนมากระบบมันก็ดึง commit message มาใส่ PR ไม่ใช่เหรอครับ?

ขึ้นอยู่กับว่าเราตั้งให้

ZiiT Sat, 03/08/2019 - 15:20

ขึ้นอยู่กับว่าเราตั้งให้ Merge PR เข้า Master branch แบบไหนครับ

อันนี้ตัวอย่างของ Github https://help.github.com/en/articles/about-merge-methods-on-github

อันนี้ของตัวอย่าง Azure DevOps https://devblogs.microsoft.com/devops/pull-requests-with-rebase/

หมายถึงว่ามันจะดึง commit

whitebigbird Sat, 03/08/2019 - 17:20

หมายถึงว่ามันจะดึง commit message มารอเอาไว้ก่อนที่จะกด create PR น่ะครับ แล้วเราก็สามารถจะ edit หรือใช้ commit message นั้นๆ ไปเลยก็ได้

แต่คิดไปคิดมา แบบที่คุณทำอ่ะเป็น practice ที่แม่นสุดแล้วครับ

ขึ้นอยู่กับ VCS hosting แต่ละเจ้าครับ ถ้า Bitbucket จะดึง commit msg มาเป็นค่า default ของ PR description แต่ถ้า Github จะไม่มีครับ ยกเว้นกำหนด .github/pull_request_template.md ไว้ใน root directory ก็จะใช้ template นั้นแทน

เรียกว่า squash merge นี่เอง

tg-thaigamer Sat, 03/08/2019 - 12:27

เรียกว่า squash merge นี่เอง ปกติ ใช้ Merge pull request แล้วคอมเม้นต์คนอื่นทำมาหมดเลย ทำให้ตัว master ปนเปื้อนจริงๆ