Phabricator Arkanist arc land vs arc ändern

Ich bin mit Phabricator und Arkanist für code-reviews in meinem team. Die 'arc-land" - Befehl ist genial, aber es ist ein Szenario, wo es nicht für uns arbeiten.

Haben wir eine einzelne xml-Datei, die enthält eine verknüpfte Liste (jedes element bezieht sich auf einen vorherigen element). Wir machen keine änderungen an dieser Datei sehr oft, aber einmal in eine Weile wir. Wenn zwei Leute änderungen an der gleichen Zeit einen "stillen" Konflikt Auftritt, was bedeutet, dass die verlinkte Liste ist gebrochen, weil sowohl die neuen Elemente, die Verknüpfung zu dem vorherigen element. Dies ist nicht sehr schwer zu beheben sind. Aber keine merge-Konflikte ausgelöst werden, die von git.

So, wenn wir laufen, arc land, die falsche xml-Datei wird automatisch geschoben. Das wollen wir nicht.

Wäre die richtige Aktion zu verwenden arc ändern, und beheben Sie dann, dass der Konflikt manuell, gefolgt von einem git-push (wie wir es heute tun, ohne jede Hektik), oder wie würden Sie vorschlagen, bewegen uns mit dieser?

InformationsquelleAutor Jay Pete | 2013-07-11
  1. 8

    Einige mögliche Ideen:

    • Können Sie arc land --hold zu stoppen, bevor die git push ausgeführt wird, und überprüfen Sie die änderungen, und führen dann git push manuell.
    • Sie können einen lokalen Git-pre-commit-hook Validierung der XML-Datei vor Git ermöglicht das Begehen.
    • Können Sie ein pre-receive hook auf dem server zum validieren der XML-Datei, bevor Sie es gedrückt, um die Fernbedienung.
    • Die Sie überschreiben können: Git merge-Verhalten mit einem merge Richtlinie in gitattributes, und ersetzen Sie das Standard-merge-mit einer, die richtig (oder nicht Zusammenführen).
    • Könnten Sie versuchen, ersetzen Sie die XML-Datei mit Daten in einem bestimmten format, das nicht über diese unerwünschten merge-Eigenschaften, denn dieses problem ist ein generelles.
    • Könnte man die Kraft verschmelzen zu scheitern, indem einige string in die Datei, die würden immer Konflikt. Zum Beispiel, fügen Sie ein Attribut wie lastNode="whatever" auf das container-element (d.h., immer auf der Linie 1 oder was auch immer), so dass zwei Bearbeitungen würde immer wieder Konflikte auf, die Linie durch die Benennung unterschiedlicher Letzte Knoten (stellen sicher, dass dies bearbeitet wurde, konnte man überprüfen, dass die lastNode richtig war, sich zur Laufzeit). Wenn die Datei wird automatisch generiert, man könnte einfach eine zufällige Zahl in einem Kommentar auf einer bekannten Linie.
    • Genial! arc land --hold ist, was Sinn macht, jetzt. Ein pre-receive hook (zu viele verschiedene lokale setups, sich an einem pre-commit-hook) validiert die XML-Datei ist der logische nächste Schritt. Dies nie ein großes problem vor, da die Datei manuell bearbeitet und leicht verstanden und behoben, durch die auch neue Mitarbeiter ohne jede Einleitung. Aber mit arc land sehe ich Menschen, das zu vergessen.
    InformationsquelleAutor Evan Priestley
Schreibe einen Kommentar