Hallo!
Wie geht man bei Code-Reviews nach einem Pull Request via GIT, bei dem man als Reviewer gesetzt worden ist am Besten vor?
Natürlich kann man sich alle Changes im Browser ansehen - z.B. auf gitHub, BitBuckets, oder was man da auch immer intern verwendet.
Jedoch ist das oft nicht so angenehm und übersichtlich und man kann sich nirgends "reinklicken", wie in der IDE.
In der IDE selber kann man schon einen checkout auf den zu reviewenden Branch machen und dann via Git History einzelne Klassen vor und nach einzelnen Commits ansehen.
Aber was hier fehlt ist eine Art Stand vor dem Pull Request des gesamten Projektes (alle Klassen) im Vergleich mit dem Stand nach dem Pull Request.
Wie macht ihr das in der Praxis?
Evtl. einen eigenen Workspace für Code-Reviews anlegen und dort auf den zu reviewenden Branch auschecken ( = Stand Projekt nach allen Commits des zu reviewenden PRs) und die Situation dort mit dem Stand im eigenem Workspace vergleichen, also dem Stand ohne den Commits, die zum PR gehören.
Wobei dann nicht die Unterschiede "gehigh-lighted" werden.
Mich würde einfach interessieren, wie ihr das in der Praxis am Besten macht, weil von Euch sicher einige bereits in Rollen sind/waren, bei denen man viel Code-Review-Erfahrung sammelt und ggf. gute Tipps geben kann.^^
Lg
Zrebna
Wie geht man bei Code-Reviews nach einem Pull Request via GIT, bei dem man als Reviewer gesetzt worden ist am Besten vor?
Natürlich kann man sich alle Changes im Browser ansehen - z.B. auf gitHub, BitBuckets, oder was man da auch immer intern verwendet.
Jedoch ist das oft nicht so angenehm und übersichtlich und man kann sich nirgends "reinklicken", wie in der IDE.
In der IDE selber kann man schon einen checkout auf den zu reviewenden Branch machen und dann via Git History einzelne Klassen vor und nach einzelnen Commits ansehen.
Aber was hier fehlt ist eine Art Stand vor dem Pull Request des gesamten Projektes (alle Klassen) im Vergleich mit dem Stand nach dem Pull Request.
Wie macht ihr das in der Praxis?
Evtl. einen eigenen Workspace für Code-Reviews anlegen und dort auf den zu reviewenden Branch auschecken ( = Stand Projekt nach allen Commits des zu reviewenden PRs) und die Situation dort mit dem Stand im eigenem Workspace vergleichen, also dem Stand ohne den Commits, die zum PR gehören.
Wobei dann nicht die Unterschiede "gehigh-lighted" werden.
Mich würde einfach interessieren, wie ihr das in der Praxis am Besten macht, weil von Euch sicher einige bereits in Rollen sind/waren, bei denen man viel Code-Review-Erfahrung sammelt und ggf. gute Tipps geben kann.^^
Lg
Zrebna