Review Board » History » Revision 2
Revision 1 (Αλέξανδρος Διαμαντίδης, 21/01/2014 15:42) → Revision 2/3 (Αλέξανδρος Διαμαντίδης, 27/01/2014 13:20)
h1. Review Board
Για το review σημαντικών αλλαγών στον κώδικα πριν την εισαγωγή του στη βασική γραμή ανάπτυξης, έχει εγκατασταθεί η εφαρμογή Review Board στην παρακάτω διεύθυνση:
http://reviews.openeclass.org/
Το κάθε μέλος της ομάδας που επιθυμεί να ανεβάσει κώδικα ή να συμμετάσχει στο review θα πρέπει να εγγραφεί στην εφαρμογή. Γενικά ακολουθούμε διαδικασία post-commit review, δηλαδή η ανάπτυξη του κώδικα θα πρέπει να γίνει πρώτα commit σε κάποιο branch και βάσει αυτού να αποσταλεί στο Review Board. Δείτε στο τέλος αυτής της σελίδας οδηγίες για την ενεργοποίηση και τις ρυθμίσεις του Mercurial Review Board Extension.
Αναλυτικό εγχειρίδιο χρήστη: http://www.reviewboard.org/docs/manual/1.7/users/
Ακολουθεί ένα παράδειγμα: ας πούμε ότι αναπτύσσουμε ένα νέο υποσύστημα στο branch "newfeature", το οποίο βασίζεται στο default branch. Αρχικά, πριν την αποστολή των αλλαγών μας για review, θα πρέπει να ελαχιστοποιήσουμε τις αλλαγές από το βασικό κώδικα που δε σχετίζονται με τις αλλαγές μας, κάνοντας merge με το default:
<pre>$ hg update newfeature
$ hg merge default
$ hg commit</pre>
Στη συνέχεια, μπορούμε να στείλουμε στο Review Board το patch που περιλαμβάνει τις αλλαγές του branch μας ως προς το default:
<pre>$ hg postreview --parent default
Changeset 6535:5df59fedd8a5
---------------------------
Merge with default.
reviewboard: http://reviews.openeclass.org/
username: adia
password: **********
Looking for 'reviews.openeclass.org /' cookie in /home/adia/.post-review-cookies.txt
Loaded valid cookie -- no login required
Repositories:
[1] Open eClass
repository id: 1
review request draft saved: http://reviews.openeclass.org/r/8/</pre>
Στη διεύθυνση που εμφανίζεται στο τέλος των μηνυμάτων, μπορούμε να βρούμε την αίτηση review που θέλουμε να κοινοποιήσουμε. Την ανοίγουμε, ελέγχουμε ότι το diff έχει όντως τις αλλαγές που θέλουμε, και συμπληρώνουμε τα επιθυμητά πεδία. Θα πρέπει να βεβαιωθούμε ότι τουλάχιστον τα Summary και Description έχουν συμπληρωθεί με την περιγραφή των αλλαγών που θέλουμε να στείλουμε, ενώ στο πεδίο Groups συμπληρώνουμε υποχρεωτικά "eclass" ώστε το αίτημα να σταλεί στη λίστα eclass-dev. Μπορούμε να συμπληρώσουμε και ονόματα συγκεκριμένων μελών εφόσον επιθυμούμε προσωπική κοινοποίηση. Τέλος, μπορούμε να δημοσιεύσουμε το αίτημα.
Κατά τη διάρκεια του ελέγχου, μπορεί να γίνουν παρατηρήσεις που οδηγούν σε αλλαγές του κώδικα που επιθυμούμε να στείλουμε. Μπορούμε να κάνουμε τις αλλαγές commit στο branch όπου δουλεύουμε, και να στείλουμε ενημερωμένο αίτημα review:
<pre>$ hg postreview --parent default -e 8</pre>
Η παράμετρος -e έχει το id που εμφανίζεται στο url του αιτήματος .
Μετά την ολοκλήρωση του ελέγχου και την έγκριση για merge, μπορούμε να στείλουμε τις αλλαγές μας στο default branch:
<pre>$ hg update default
$ hg merge newfeature
$ hg commit</pre>
Αν το branch προοριζόταν για συγκεκριμένο feature που η ανάπτυξή του έχει ολοκληρωθεί, στο commit μπορούμε να κλείσουμε το συγκεκριμένο branch δίνοντας την παράμετρο @--close-branch@ στην εντολή του commit.
h2. Εγκατάσταση Review Board Extension
Το "Review Board Extension":http://mercurial.selenic.com/wiki/ReviewboardExtension του Mercurial μπορεί να εγκατασταθεί ως εξής:
<pre>$ mkdir -p ~/.hgext
$ cd ~/.hgext
$ hg clone https://bitbucket.org/ccaughie/hgreviewboard</pre>
Στη συνέχεια, εισάγουμε στο αρχείο @~/.hgrc@ τα παρακάτω:
<pre>[extensions]
reviewboard = ~/.hgext/hgreviewboard
[reviewboard]
server = http://reviews.openeclass.org
user = <username μας στο Review Board>
password = <password στο Review Board></pre>
Μετά τα παραπάνω, θα είναι διαθέσιμη η εντολή @hg postreview@.
Για το review σημαντικών αλλαγών στον κώδικα πριν την εισαγωγή του στη βασική γραμή ανάπτυξης, έχει εγκατασταθεί η εφαρμογή Review Board στην παρακάτω διεύθυνση:
http://reviews.openeclass.org/
Το κάθε μέλος της ομάδας που επιθυμεί να ανεβάσει κώδικα ή να συμμετάσχει στο review θα πρέπει να εγγραφεί στην εφαρμογή. Γενικά ακολουθούμε διαδικασία post-commit review, δηλαδή η ανάπτυξη του κώδικα θα πρέπει να γίνει πρώτα commit σε κάποιο branch και βάσει αυτού να αποσταλεί στο Review Board. Δείτε στο τέλος αυτής της σελίδας οδηγίες για την ενεργοποίηση και τις ρυθμίσεις του Mercurial Review Board Extension.
Αναλυτικό εγχειρίδιο χρήστη: http://www.reviewboard.org/docs/manual/1.7/users/
Ακολουθεί ένα παράδειγμα: ας πούμε ότι αναπτύσσουμε ένα νέο υποσύστημα στο branch "newfeature", το οποίο βασίζεται στο default branch. Αρχικά, πριν την αποστολή των αλλαγών μας για review, θα πρέπει να ελαχιστοποιήσουμε τις αλλαγές από το βασικό κώδικα που δε σχετίζονται με τις αλλαγές μας, κάνοντας merge με το default:
<pre>$ hg update newfeature
$ hg merge default
$ hg commit</pre>
Στη συνέχεια, μπορούμε να στείλουμε στο Review Board το patch που περιλαμβάνει τις αλλαγές του branch μας ως προς το default:
<pre>$ hg postreview --parent default
Changeset 6535:5df59fedd8a5
---------------------------
Merge with default.
reviewboard: http://reviews.openeclass.org/
username: adia
password: **********
Looking for 'reviews.openeclass.org /' cookie in /home/adia/.post-review-cookies.txt
Loaded valid cookie -- no login required
Repositories:
[1] Open eClass
repository id: 1
review request draft saved: http://reviews.openeclass.org/r/8/</pre>
Στη διεύθυνση που εμφανίζεται στο τέλος των μηνυμάτων, μπορούμε να βρούμε την αίτηση review που θέλουμε να κοινοποιήσουμε. Την ανοίγουμε, ελέγχουμε ότι το diff έχει όντως τις αλλαγές που θέλουμε, και συμπληρώνουμε τα επιθυμητά πεδία. Θα πρέπει να βεβαιωθούμε ότι τουλάχιστον τα Summary και Description έχουν συμπληρωθεί με την περιγραφή των αλλαγών που θέλουμε να στείλουμε, ενώ στο πεδίο Groups συμπληρώνουμε υποχρεωτικά "eclass" ώστε το αίτημα να σταλεί στη λίστα eclass-dev. Μπορούμε να συμπληρώσουμε και ονόματα συγκεκριμένων μελών εφόσον επιθυμούμε προσωπική κοινοποίηση. Τέλος, μπορούμε να δημοσιεύσουμε το αίτημα.
Κατά τη διάρκεια του ελέγχου, μπορεί να γίνουν παρατηρήσεις που οδηγούν σε αλλαγές του κώδικα που επιθυμούμε να στείλουμε. Μπορούμε να κάνουμε τις αλλαγές commit στο branch όπου δουλεύουμε, και να στείλουμε ενημερωμένο αίτημα review:
<pre>$ hg postreview --parent default -e 8</pre>
Η παράμετρος -e έχει το id που εμφανίζεται στο url του αιτήματος .
Μετά την ολοκλήρωση του ελέγχου και την έγκριση για merge, μπορούμε να στείλουμε τις αλλαγές μας στο default branch:
<pre>$ hg update default
$ hg merge newfeature
$ hg commit</pre>
Αν το branch προοριζόταν για συγκεκριμένο feature που η ανάπτυξή του έχει ολοκληρωθεί, στο commit μπορούμε να κλείσουμε το συγκεκριμένο branch δίνοντας την παράμετρο @--close-branch@ στην εντολή του commit.
h2. Εγκατάσταση Review Board Extension
Το "Review Board Extension":http://mercurial.selenic.com/wiki/ReviewboardExtension του Mercurial μπορεί να εγκατασταθεί ως εξής:
<pre>$ mkdir -p ~/.hgext
$ cd ~/.hgext
$ hg clone https://bitbucket.org/ccaughie/hgreviewboard</pre>
Στη συνέχεια, εισάγουμε στο αρχείο @~/.hgrc@ τα παρακάτω:
<pre>[extensions]
reviewboard = ~/.hgext/hgreviewboard
[reviewboard]
server = http://reviews.openeclass.org
user = <username μας στο Review Board>
password = <password στο Review Board></pre>
Μετά τα παραπάνω, θα είναι διαθέσιμη η εντολή @hg postreview@.