Nächste: , Vorige: , Nach oben: Mitwirken   [Inhalt][Index]


22.13 Beiträge von anderen überprüfen

Den wohl größten Nutzen, damit GNU Guix als Projekt wächst, steuern Sie bei, wenn Sie die Beiträge von anderen gegenlesen und überprüfen. Dazu müssen Sie kein Committer sein; Sie wenden den Patch an (git am bzw. git apply), lesen den Quellcode, erstellen ihn und setzen ihn Linting aus, außerdem können Sie die Patchreihen anderer Leute auf Ihrem Rechner ausprobieren und kommentieren, wie Sie zurechtkommen, was anderen Committern Sicherheit gibt. Vereinfacht gesagt stellen Sie sicher, dass der Prüfliste aus dem Abschnitt Einreichen von Patches korrekt Folge geleistet wurde. Wenn eine Patchserie bereits überprüft worden ist, dann hat sie die besten Chancen, schneller ins Repository gemerget zu werden. Wenn Sie also gerne eine Änderung angenommen sehen würden und diese noch niemand überprüft hat, dann übernehmen am besten Sie die Aufgabe!

Review comments should be unambiguous; be as clear and explicit as you can about what you think should be changed, ensuring the author can take action on it. Please try to keep the following guidelines in mind during review:

  1. Be clear and explicit about changes you are suggesting, ensuring the author can take action on it. In particular, it is a good idea to explicitly ask for new revisions when you want it.
  2. Remain focused: do not change the scope of the work being reviewed. For example, if the contribution touches code that follows a pattern deemed unwieldy, it would be unfair to ask the submitter to fix all occurrences of that pattern in the code; to put it simply, if a problem unrelated to the patch at hand was already there, do not ask the submitter to fix it.
  3. Ensure progress. As they respond to review, submitters may submit new revisions of their changes; avoid requesting changes that you did not request in the previous round of comments. Overall, the submitter should get a clear sense of progress; the number of items open for discussion should clearly decrease over time.
  4. Aim for finalization. Reviewing code is time-consuming. Your goal as a reviewer is to put the process on a clear path towards integration, possibly with agreed-upon changes, or rejection, with a clear and mutually-understood reasoning. Avoid leaving the review process in a lingering state with no clear way out.
  5. Review is a discussion. The submitter’s and reviewer’s views on how to achieve a particular change may not always be aligned. To lead the discussion, remain focused, ensure progress and aim for finalization, spending time proportional to the stakes51. As a reviewer, try hard to explain the rationale for suggestions you make, and to understand and take into account the submitter’s motivation for doing things in a certain way.

When you deem the proposed change adequate and ready for inclusion within Guix, the following well understood/codified ‘Reviewed-by: Your Name <your-email@example.com>52 line should be used to sign off as a reviewer, meaning you have reviewed the change and that it looks good to you:

If you are not a committer, you can help others find a series you have reviewed more easily by adding a reviewed-looks-good usertag for the guix user (siehe Debbugs-Usertags).


Fußnoten

(51)

The tendency to discuss minute details at length is often referred to as “bikeshedding”, where much time is spent discussing each one’s preference for the color of the shed at the expense of progress made on the project to keep bikes dry.

(52)

The ‘Reviewed-by’ Git trailer is used by other projects such as Linux, and is understood by third-party tools such as the ‘b4 am’ sub-command, which is able to retrieve the complete submission email thread from a public-inbox instance and add the Git trailers found in replies to the commit patches.


Nächste: Das Guix-Paket aktualisieren, Vorige: Commit-Zugriff, Nach oben: Mitwirken   [Inhalt][Index]