Dokumentation

Ergebnis überprüfen und zusammenführen

Was auf dem pull request eintrifft, was zu prüfen ist und wie Arbeit zurückgesendet wird.

Coroid erstellt abschließend einen eigenen pull request auf einem separaten Branch gegenüber Ihrem Basis-Branch. Ein Merge erfolgt nicht – diese Entscheidung bleibt bei Ihnen.

Was damit geliefert wird

  • die Code-Änderung
  • dazu geschriebene oder aktualisierte Tests
  • das Ergebnis des Ausführens Ihrer bestehenden Test-Suite
  • das Ergebnis der von Ihrer Organisation konfigurierten Qualitätsprüfungen
  • eine Überprüfung der Differenzen, wobei bereits behobene oder aufgelistete Probleme enthalten sind

Es geht nicht darum, Ihre Überprüfung zu ersetzen – vielmehr soll Ihre Überprüfung nicht die erste sein.

Was konkret zu prüfen ist

Die mechanischen Aspekte wurden bereits überprüft. Konzentrieren Sie sich auf die Punkte, die nur Sie beurteilen können:

  1. Löst es das richtige Problem? Vergleichen Sie es mit den Akzeptanzkriterien aus der Spezifikation – nicht mit Ihrer Erinnerung daran, was Sie angefordert haben.
  2. Passt es zum vorhandenen Codebestand? Konventionen, Benennungen, die Struktur der Abstraktion. Coroid analysiert Ihren Code, um diese zu erkennen – meist korrekt, aber nicht immer.
  3. Welche Bereiche werden berührt, die Sie nicht erwartet haben? Schauen Sie sich die Dateiliste vor der Differenzansicht an.
  4. Sind die neuen Tests sinnvoll? Ein bestehender Test, der nichts beweist, ist schlimmer als gar kein Test.

Arbeit zurücksenden

Sie haben drei Optionen – je nach Aufwand steigend:

  • Kommentar hinzufügen und Nachbesserung anfordern — der Auftrag wird mit Ihren Hinweisen an den Entwickler-Agenten zurückgesendet und behält seinen Kontext bei. Ideal für den Fall „Das ist richtig, aber unvollständig“.
  • Plan ablehnen und erneut ausführen — wenn der Ansatz falsch ist, nicht die Umsetzung.
  • Auftrag abbrechen — wenn die Arbeit überhaupt nicht erfolgen soll. Das Abbrechen stoppt den Prozess und gibt den Agenten-Slot frei.

Seien Sie bei Nachbesserungskommentaren genauso präzise wie bei einem Kollegen. „Dies behandelt den leeren Fall in parseRange" liefert eine Lösung; „braucht Arbeit“ bleibt eine Vermutung.

Zusammenführen

Führen Sie den Merge über Ihren eigenen Anbieter durch – unter Verwendung Ihrer üblichen Branch-Schutzmechanismen, erforderlichen Prüfungen und Freigaberegeln. Coroid öffnet den pull request; Ihr bestehender Prozess bestimmt, was damit geschieht.

Am Zusammenführen ist nichts Besonderes – das ist beabsichtigt. Der pull request ist ein gewöhnlicher Pull Request und durchläuft somit Ihre vertrauten Überprüfungs- und CI-Prozesse.

Weiter

Wohin Sie als Nächstes gehen können — führt je nach gewünschter Aktion zum Rest der Dokumentation.