Pull Request - trabnm1313/SDTP GitHub Wiki

What is Pull Request?

การทำ Pull Request

ก่อนที่จะทำการ merge branch เราจะทำ Pull Request ทุกครั้ง โดยที่จะเลือกสมาชิกในทีม 1 คนเป็น Reviewer(ซึ่งคนในทีมที่จะรับหน้าที่นี้ก็คือ trabnm1313) คอยทำหน้าที่ตรวจสอบความถูกต้องต่างๆของ Code ที่จะ Merge กลับเข้าไปที่ Branch หลักก่อนที่แยกออกมา

ขั้นตอนทำ Pull Request

README.md in branch main after merge branch
                   Image : README.md ใน branch main

README.md in branch main after merge branch
                   Image : README.md ใน branch develop

เมื่อต้องการจะ Merge Branch Develop กลับไป Branch Main จะไปสร้าง Pull Request และใส่รายละเอียดและกำหนด Reviewer, Assignee, Labels

Reviewer

  • คือ ผู้ที่จะมาตรวจสอบไฟล์ต่างๆก่อนจะ Merge Branch

Assignee

  • คือ ผู้จะเป็นคนแก้ไขเมื่อ Pull Request นั้นมีปัญหาแล้วถูกตีกลับให้ไปแก้ไข

Labels

  • คือ Tag ที่บอกว่า Pull Request นี้เกี่ยวกับอะไร เช่น เพิ่มfeatureใหม่,แก้bug

หลังจากที่ Reviewer ได้ตรวจสอบความถูกต้องของไฟล์ที่มีการเปลี่ยนแปลงแล้ว Reviewer จะเป็นคนกด Merge branch เอง

Open Pull requests
                            Image : Open Pull Request

หลังจากที่ Reviewer ได้ตรวจสอบความถูกต้องของไฟล์ที่มีการเปลี่ยนแปลงแล้ว Reviewer จะเป็นคนกด Merge branch เอง Pull requests                             Image : Pull Request

Pull requests pass                 Image : README.md ใน branch main หลังจาก merge branch

แต่ถ้า Reviewer ตรวจสอบแล้วพบว่า Pull Request นั้นไม่ผ่านจะมีการ Request Change ให้กลับไปแก้มาใหม่ก่อนจนกว่าจะผ่าน

Pull requests Change requested
                         Image : Pull requests ไม่ผ่าน