การรีวิว pull request
อินซิเดนต์ส่วนใหญ่เริ่มจากการเปลี่ยนแปลง Polylane รีวิวทุก pull request ใน repository ที่เชื่อมต่อไว้เทียบกับทรัพยากร production ที่มัน deploy ไป และโพสต์ความเห็นเดียวที่บอกว่าผ่านหรือไม่ผ่าน
ทำอะไรได้
เอเจนต์อ่าน diff และเมตริก log และ issue ที่เปิดอยู่แบบเรียลไทม์ของทรัพยากรที่ repository deploy ไป แล้วตัดสินว่าการเปลี่ยนแปลงนั้นทำร้าย production ได้หรือไม่ คุณได้รับความเห็นเดียวบน pull request check Polylane production impact ที่เลือกใช้ได้ และเธรดในคอนโซลพร้อมการวิเคราะห์ฉบับเต็ม
ทำงานอย่างไร
- การเปิด pull request ไปยัง branch เริ่มต้น การ push ไปยังมัน หรือการทำเครื่องหมายว่าพร้อมรีวิว จะเริ่มการรีวิว commit ล่าสุดของมัน draft จะไม่ถูกรีวิว
- การ push ครั้งถัดไปแทนที่การรีวิวที่กำลังดำเนินอยู่ ดังนั้นมีเพียง commit ล่าสุดเท่านั้นที่ถูกตัดสิน
- การเปลี่ยนแปลงที่แตะเฉพาะเอกสาร การทดสอบ หรือไฟล์ CI จะผ่านโดยไม่ต้องสืบสวน
- Polylane ตามเส้นเชื่อม deploy ของ repository ไปยังทรัพยากร production ของมัน หรือรีวิวเทียบกับบัญชีคลาวด์ของ workspace ทั้งหมดเมื่อไม่มีเส้นเชื่อม
- เอเจนต์สืบสวนในเธรดและบันทึกคำตัดสิน
passหรือfail - ความเห็นหนึ่งรายการต่อ pull request ถูกแก้ไขในแต่ละครั้งที่รีวิว กรณีไม่ผ่านจะอ่านว่า Hold this merge พร้อมระดับผลกระทบและข้อค้นพบ กรณีผ่านจะอ่านว่า Production impact unlikely หรือบอกว่าการแก้ไข issue ที่ตรวจพบคาดว่าจะแก้ปัญหาได้หรือไม่ เมื่อการ push ครั้งถัดไปจัดการข้อกังวลแล้ว ความเห็นจะพลิกกลับเป็นผ่านและบอกไว้เช่นนั้น
เอเจนต์ยังเสนอการปรับปรุงด้าน observability ได้ ในรูปการรีวิวชุดเดียวที่มีความเห็นแนะนำหลายรายการ หรือ pull request แบบ stacked ข้อเสนอเหล่านี้ไม่มีผลต่อคำตัดสิน
ดำเนินการกับการรีวิว
กล่าวถึง @polylane ในความเห็นบน pull request แล้ว Polylane จะตอบในการสนทนาเดียวกัน ผู้ที่ไม่ใช่สมาชิกจะได้รับข้อความเกี่ยวกับสิทธิ์เข้าถึงแทน
| คุณขอ | สิ่งที่เกิดขึ้น |
|---|---|
| คำถาม | เอเจนต์ตอบจากเธรดรีวิว โดยมีหลักฐานอยู่ตรงหน้า |
| ดูอีกครั้ง | เอเจนต์รีวิวการ push ล่าสุดอีกครั้งและพิจารณาคำตัดสินใหม่ |
| การแก้ไข | Polylane เริ่ม fix run และเปิด pull request แบบ stacked ไปยัง branch ของ pull request นั้น โดยลิงก์จากความเห็น |
แท็บ Changes ของ repository ในคอนโซลแสดงทุกการรีวิว กรณีไม่ผ่านจะมี Fix with Polylane ซึ่งเริ่ม fix run เดียวกัน และ Dismiss เมื่อข้อกังวลไม่เกี่ยวข้อง: pull request ยังคงถูกรีวิวต่อไปแต่จะไม่ได้รับความเห็นเพิ่มจนกว่าคุณ Restore มัน
| สถานะ | คำอธิบาย |
|---|---|
| In review | Polylane กำลังรีวิวการ push ล่าสุด |
| Flagged | การรีวิวพบข้อกังวลด้าน production |
| Passed | ไม่พบข้อกังวลด้าน production |
| Resolved | การ push ครั้งถัดไปจัดการข้อกังวลแล้ว |
| Dismissed | สมาชิกปัดตกการรีวิว pull request จะไม่ได้รับความเห็นเพิ่ม |
กำหนดค่า
การรีวิวเปิดอยู่สำหรับทุก repository ที่เชื่อมต่อไว้ Settings > Pull request reviews ปิดการรีวิวทั้ง workspace และกำหนดว่าเธรดรีวิวจะแชร์กับผู้ที่ดู pull request หรือไม่ ใต้ Settings > Repositories แท็บ Settings ของ repository เพิ่มอีกสามช่อง
| ช่อง | คำอธิบาย |
|---|---|
| Review pull requests for production impact | ปิดการรีวิวสำหรับ repository นี้เท่านั้น |
| Pull request review instructions | ข้อความอิสระที่เอเจนต์อ่านในทุกการรีวิว: path ที่สำคัญต่อการ deploy หรือข้อกังวลที่ด่านอื่นจัดการอยู่แล้ว |
| Block merging on production concerns | โพสต์ check Polylane production impact บนทุก pull request ที่รีวิว ซึ่งจะไม่ผ่านเมื่อพบข้อกังวล เปิดเป็นค่าเริ่มต้น กำหนดให้เป็นข้อบังคับใน branch protection เพื่อบล็อกการ merge |
check นี้ยังเสร็จสมบูรณ์บน commit ใน merge queue ด้วย เนื่องจาก pull request ถูกรีวิวก่อนเข้าคิว คำตัดสินไม่ผ่านส่งอีเมล Production Concern Found ได้ ซึ่งปิดเป็นค่าเริ่มต้นใต้การแจ้งเตือน
หมายเหตุเกี่ยวกับแผน
แต่ละแผนรวมจำนวนการรีวิว pull request ต่อเดือน โควตาของแผน Free ระบุไว้ในการเรียกเก็บเงินและการใช้งาน และสมาชิกจะได้รับอีเมลเมื่อโควตาใกล้หมด เมื่อเกินโควตา คอนโซลจะบันทึก pull request เป็น Production impact not verified และไม่มีอะไรถูกโพสต์ไปยัง GitHub
ที่เกี่ยวข้อง
- Repository เพื่อเชื่อมต่อ repository และเชื่อมโยงทรัพยากรของมัน
- Fix run สำหรับวิธีที่เธรดรีวิวไปถึงคำตัดสิน
- Autofix สำหรับ pull request ที่ fix run เปิด
- การแจ้งเตือน สำหรับอีเมลที่การรีวิวที่ไม่ผ่านส่งได้
Issues
ทุกปัญหาที่ Polylane ตรวจพบหรือได้รับจากระบบ alert ของคุณจะกลายเป็น issue หนึ่งรายการ ซึ่งจัดการโดย fix run หนึ่งรายการที่เธรดของมันเล่าเรื่องทั้งหมด
Advisories
คำแนะนำต่อทรัพยากรสำหรับการกำหนดค่าผิด ความเสี่ยงด้านความทนทาน และช่องว่างด้าน observability แต่ละรายการมาพร้อมการแก้ไขที่คุณส่งให้เอเจนต์ได้