จะตั้งค่าการตรวจสอบโค้ดโดยใช้ Gitlab ได้อย่างไร?


86

จะตั้งค่าการตรวจสอบโค้ดโดยใช้ Gitlab ได้อย่างไร? ฉันเห็นว่ามันแสดงเป็นฟีเจอร์บนเว็บไซต์ Gitlab แต่ดูเหมือนจะไม่พบคำแนะนำในการตั้งค่า (สำหรับเรื่องนั้นลิงก์ไปยังคู่มือผู้ใช้ Gitlab จะได้รับการชื่นชมมากที่สุด)

การค้นหาบางส่วนของฉันระบุว่า 'คำขอรวม' เป็นวิธีที่จะไป ... แต่ฉันพบว่ามีข้อ จำกัด คำขอการรวมที่ออกจะแสดงการกระทำทั้งหมดระหว่างสาขาหนึ่งและอีกสาขาหนึ่ง ดูเหมือนว่าฉันจะสามารถดูความแตกต่างที่สร้างขึ้นสำหรับแต่ละคอมมิตเท่านั้น ตัวอย่างเช่นสมมติว่าฉันมีไฟล์ที่ต้องการตรวจสอบ เป็นไฟล์ใหม่ แต่ฉันได้ส่งการเปลี่ยนแปลงไปแล้วมากกว่า 10 คอมมิตในสาขา dev ถ้าฉันออกคำขอรวมสำหรับสาขา dev นั้นจากการรวมฉันเห็น 10 คอมมิตซึ่งแต่ละรายการแสดงการเปลี่ยนแปลงที่เพิ่มขึ้นในไฟล์ ... ฉันต้องการตรวจสอบสิ่งทั้งหมด ใหม่!

ฉันเห่าต้นไม้ผิดที่นี่หรือเปล่า? มีเครื่องมือตรวจสอบโค้ดจริงที่ฉันสามารถใช้ใน GitLab ได้หรือไม่หรือกำลังขอการผสานระหว่างทางและฉันกำลังใช้มันอย่างไม่ถูกต้อง วิธีใดดีที่สุดในการตั้งค่าการตรวจสอบโค้ดที่เหมาะสมที่นี่


2
GitLab 6.4 และมุมมองที่แตกต่างแบบเคียงข้างกันสามารถช่วยในการตรวจสอบโค้ดได้: ดูคำตอบของฉันด้านล่าง
VonC

1
ด้วย GitLab 13.1 (มิถุนายน 2020) ตอนนี้คุณมี Merge Request Reviews ดูคำตอบที่แก้ไขของฉันด้านล่าง
VonC

คำตอบ:


25

หมายเหตุ: ตั้งแต่ GitLab 6.4 สามารถใช้มุมมองต่างแบบเคียงข้างกันได้โปรดดูที่ " คำขอดึง 5308 "

(กรกฎาคม 2013)ยังไม่มีความเป็นไปได้ที่จะแสดงความคิดเห็นในแต่ละบรรทัด แต่ในระดับไฟล์เท่านั้น
Daniel Sokolowskiกล่าวถึงในความคิดเห็นที่รองรับการแสดงความคิดเห็นต่อบรรทัดแล้ว (09/2014):

สมาชิกในทีมของคุณสามารถแสดงความคิดเห็นเกี่ยวกับคำขอรวมโดยทั่วไปหรือในบางบรรทัดด้วยความคิดเห็นทางไลน์

สิ่งนี้ยังช่วยได้สำหรับกิจกรรมการตรวจสอบโค้ด

https://f.cloud.github.com/assets/4224518/1558702/e0fe633a-4fa3-11e3-9388-3f3e445cb6d4.png


6 ปีต่อมาสำหรับGitLab 13.1 (มิถุนายน 2020) :

การตรวจสอบคำขอรวมถูกย้ายไปที่ Core

เดิมเปิดตัวใน GitLab 11.4 เป็นฟีเจอร์ GitLab Premium การตรวจสอบคำขอการผสานช่วยให้ผู้ตรวจสอบคำขอรวมสามารถ:

  • ส่งความคิดเห็นหลายรายการพร้อมกัน
  • ลดเสียงการแจ้งเตือนสำหรับผู้เขียนคำขอรวมและ
  • ช่วยให้กระบวนการตรวจสอบมีความสอดคล้องและคล่องตัวมากขึ้น

https://about.gitlab.com/images/13_1/batch_comments.png

นับตั้งแต่เปิดตัวเราได้ประเมินตำแหน่งของมันอีกครั้งในรูปแบบการกำหนดราคาตามผู้ซื้อของเราและในฐานะส่วนหนึ่งของ 13.1 เรารู้สึกตื่นเต้นที่จะประกาศว่าฟีเจอร์นี้ได้ย้ายไปที่ GitLab Core แล้ว

ดูเอกสารและปัญหา


ขณะนี้ความคิดเห็นต่อบรรทัดได้รับการสนับสนุน: "สมาชิกในทีมของคุณสามารถแสดงความคิดเห็นเกี่ยวกับคำขอรวมโดยทั่วไปหรือในบางบรรทัดที่มีความคิดเห็นในบรรทัด" ( about.gitlab.com/2014/09/29/gitlab-flow )
Daniel Sokolowski

1
@DanielSokolowski เยี่ยม! ฉันได้รวมความคิดเห็นของคุณไว้ในคำตอบเพื่อให้มองเห็นได้มากขึ้น
VonC

9

ฉันทำการตรวจสอบโค้ดใน Gitlab มานานกว่าสองเดือนโดยแทบจะไม่มีแรงเสียดทาน ฉันได้ตั้งค่าrss2emailเพื่อส่งการแจ้งเตือนทางอีเมลทุกครั้งที่นักพัฒนาผลักดันการกระทำใหม่ ๆ จากนั้นฉันใช้คุณสมบัติความคิดเห็นของ Gitlab เพื่อแสดงความคิดเห็นเกี่ยวกับรหัสที่พุช

น่าเสียดายที่ Gitlab ไม่อนุญาตให้แสดงความคิดเห็นในไฟล์เองโดยเฉพาะในคอมมิต (เช่นเดียวกับ Github ฉันเดา) เมื่อใดก็ตามที่ฉันพบว่าตัวเองอยู่ในสถานการณ์ที่ต้องแสดงความคิดเห็นเกี่ยวกับสิ่งที่ฉันพลาดในการกระทำครั้งก่อนฉันใช้เครื่องมือตำหนิเพื่อค้นหาข้อตกลงที่แนะนำ / เปลี่ยนแปลงส่วนรหัสที่จะแสดงความคิดเห็น

มันยังห่างไกลจากความสมบูรณ์แบบ แต่ก็ใช้งานได้ดีจนถึงตอนนี้


1
แทนที่จะใช้ rss2email เราสามารถใช้การแจ้งเตือนของ Gitlab เพื่อรับการแจ้งเตือนเกี่ยวกับการพุช
vadipp

ฉันมีปัญหา / วิธีแก้ปัญหาเดียวกัน ฉันเชื่อว่ามันจะเป็นการเพิ่มคุณสมบัติที่ดีที่คุณสามารถเพิ่มความคิดเห็นลงในการกระทำที่ถูกต้องโดยตำหนิบรรทัดใดบรรทัดหนึ่งในมุมมองที่แตกต่างหรือไฟล์ (ฉันหมายถึงจากการเรียกดูไฟล์ในอินเทอร์เฟซเว็บหรือส่วนต่างไม่ใช่การตำหนิ)
AlejandroVD

2

คุณสามารถดูรหัสที่ส่งใน Merge Request สำหรับที่เก็บอื่นหรือในที่เก็บปัจจุบัน
ตัวอย่างhttp://demo.gitlab.com/diaspora/diaspora/commits/master

จากนั้นคุณสามารถเพิ่มความคิดเห็นในการเปลี่ยนแปลงไฟล์ที่มุ่งมั่น (ปุ่มReply) หรือการคอมมิตทั้งหมด

ตัวอย่างhttp://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

ส่งผลให้การสื่อสารเป็นตรวจสอบรหัส อย่างไรก็ตามโดยส่วนตัวแล้วฉันแนะนำให้ทำการตรวจสอบโค้ดบนพีซีเครื่องหนึ่งด้วยการสื่อสารแบบตัวต่อตัวทุกครั้งที่ทำได้และใช้เครื่องมือสำหรับบันทึกผลลัพธ์หรือเมื่อต้องการความเป็นทางการมากขึ้น

สำหรับไฟล์ revue ที่มีคอมมิตจำนวนมากเช่นhttp://demo.gitlab.com/diaspora/diaspora/blame/master/README.md ให้ ดูโดยใช้blameเพื่อทำความเข้าใจว่าใครทำอะไร อย่างไรก็ตามในมุมมองนี้ไม่มีตัวเลือกในการสื่อสารและเพิ่มความคิดเห็น ฉันอยากจะแนะนำให้เพิ่มการเปลี่ยนแปลงเป็นความคิดเห็นในกรณีนี้


7
ฉันได้รับ 404 สำหรับลิงก์แรกที่สองและสุดท้ายในคำตอบของคุณ
Bryan Oakley

1
ตามที่ระบุไว้ในโฮมเพจ demo.gitlab.com "IS A SANDBOX - จะรีเซ็ตทุกชั่วโมง" ดังนั้นตัวอย่างทั้งหมดจึงถูกลบออก นี่ไม่ใช่ยานพาหนะที่ดีสำหรับตัวอย่าง
Uriah Blatherwick

ใช่โปรดพิจารณาการตั้งค่าใหม่ด้วยตัวอย่างที่เหมาะสม คำตอบของคุณดูเหมือนจะเป็นคำแนะนำที่มั่นคง
ข้อมูล

0

ใช่. คำขอรวมเป็นวิธีที่จะทำให้การตรวจสอบของเพื่อนทำได้สำเร็จ

ควรมีแท็บ 'diff' ที่จะแสดงการเปลี่ยนแปลงของการกระทำทั้งหมด (กล่าวถึงที่นี่: http://youtu.be/DyAX8ws5OIc?t=3m2s )

นอกจากนี้วิดีโอยังอธิบายถึงวิธีการใช้เพื่อตรวจสอบโดยเพื่อน


0

กรณีการใช้งานปกติของการตรวจสอบโค้ดคือการตรวจสอบโค้ดในสาขาก่อนที่จะรวมเป็นหลักหรือที่คล้าย ฉันมีสถานการณ์ที่ฉันได้พัฒนาโครงการและต้องการให้ทุกคนในทีมตรวจสอบโค้ดทั้งหมด

สิ่งที่ฉันทำคือ:

ชำระค่าคอมมิตแรกทำการเปลี่ยนแปลงกระทำและผลักดัน

git co -b FIRST_COMMIT eb67f06c2b3222c0219214b176c41922bc454881
vi README.md
git add README.md
git ci -m "First commit modified so can get full diff against it"
git push --set-upstream origin FIRST-COMMIT

ชำระค่าคอมมิตสุดท้ายทำการเปลี่ยนแปลงกระทำและผลักดัน

git co -b master
vi README.md
git add README.md
git ci -m "Last commit modified so can get full diff against it"
git push --set-upstream origin LAST-COMMIT

บน GitLab / GitHub สร้างคำขอดึง

  • เป็นการรวมจาก LAST_COMMIT เป็น FIRST_COMMIT

ใช้ได้ผลสำหรับฉัน!


นั่นไม่ทำให้คุณมีสาขา "ขยะ" สองสาขาในที่เก็บและไม่มีการติดตามความคิดเห็นในสาขาหลักใช่หรือไม่ หากความคิดเห็นต้องการการเปลี่ยนแปลงโค้ดคุณจะรวมเข้ากับความเชี่ยวชาญหรือไม่
user2084572

ใช่จะมีสาขา FIRST_COMMIT และ LAST_COMMIT ที่ง่ายต่อการลบ ( git br --delete --force origin FIRST_COMMIT LAST_COMMIT; git br --delete --force FIRST_COMMIT LAST_COMMIT) คุณสามารถใช้ branch off master ที่แตกต่างกันเพื่อทำการเปลี่ยนแปลงหรือสร้างปัญหาแยกต่างหากด้วยตนเอง และสร้างสาขาอย่างน้อยหนึ่งสาขาในภายหลัง (เช่นหนึ่งสาขาต่อฉบับ) หากมีข้อเสนอแนะมากเกินไป
HankCa
โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.