Closed kostopoulos closed 9 years ago
Όλα τέλεια, αλλά αν θέλεις μια συμβουλή που θα σε βοηθήσει και σε άλλα open source contributions: Να αποφεύγεις να στέλνεις pull requests από το master branch του fork σου.
Ένας απλός κανόνας είναι, ποτέ δε γράφουμε απευθείας στο master branch. Το master branch του fork μας το έχουμε μόνο για να "τραβάμε" τις αλλαγές από το upstream repository.
Για κάθε αλλαγή που θέλουμε να προτείνουμε μέσω pull request, δημιουργούμε πρώτα ένα topic branch—όπως μας αρέσει να τα ονομάζουμε. Το όνομα αυτό είναι απλά ένα convention. Στην ουσία είναι ένα απλό branch από το master, το οποίο προηγουμένως το έχουμε συγχρονίσει με το upstream master με την εξής διαδικασία:
git checkout master
git fetch upstream
git merge upstream/master
git checkout -b my-topic-branch
Ως my-topic-branch
, φυσικά, επιλέγουμε ένα όνομα που να έχει νόημα και να σχετίζεται με τις αλλαγές που θέλουμε να προτείνουμε.
Στην προκειμένη περίπτωση εδώ, θα λέγαμε κάτι σαν add-member-info
.
Στη συνέχεια, όταν εγώ ή κάποιος άλλο από το core team, θα κάνει merge το pull request, θα μπορέσεις να διαγράψεις το topic branch ώστε να κρατάς τακτοποιημένο το fork σου.
Τώρα, κανονικά, θα μπορούσα να κάνω merge το παρόν pull request. Όμως, για λόγους εκπαίδευσης, θα σε ταλαιπωρήσω λίγο. Θα κλείσω το pull request αντί να το κάνω merge. Στη συνέχεια, θα σου έλεγα να διαγράψεις το fork, ώστε να μπορέσεις κάνεις fork ξανά ένα φρέσκο αντίγραφο. Θα κάνεις πάλι clone τοπικά, διαγράφοντας πρώτα το παλιό clone (ή μετακινώντας το σε άλλο path). Τέλος, θα κάνεις τις ίδιες αλλαγές, όμως αυτή τη φορά σε ένα topic branch, αντί για το master.
:smile:
Χεχε!Εντάξει.Θα το κάνω σωστά.Δεν είναι δα και τόσο δύσκολο ;-)
Στάλθηκε από το iPhone μου
28 Δεκ 2014, 12:48 μ.μ., ο/η Petros Amiridis notifications@github.com έγραψε:
Όλα τέλεια, αλλά αν θέλεις μια συμβουλή που θα σε βοηθήσει και σε άλλα open source contributions: Να αποφεύγεις να στέλνεις pull requests από το master branch του fork σου.
Ένας απλός κανόνας είναι, ποτέ δε γράφουμε απευθείας στο master branch. Το master branch του fork μας το έχουμε μόνο για να "τραβάμε" τις αλλαγές από το upstream repository.
Για κάθε αλλαγή που θέλουμε να προτείνουμε μέσω pull request, δημιουργούμε πρώτα ένα topic branch—όπως μας αρέσει να τα ονομάζουμε. Το όνομα αυτό είναι απλά ένα convention. Στην ουσία είναι ένα απλό branch από το master, το οποίο προηγουμένως το έχουμε συγχρονίσει με το upstream master με την εξής διαδικασία:
git checkout master git fetch upstream git merge upstream/master git checkout -b my-topic-branch Ως my-topic-branch, φυσικά, επιλέγουμε ένα όνομα που να έχει νόημα και να σχετίζεται με τις αλλαγές που θέλουμε να προτείνουμε.
Στην προκειμένη περίπτωση εδώ, θα λέγαμε κάτι σαν add-member-info.
Στη συνέχεια, όταν εγώ ή κάποιος άλλο από το core team, θα κάνει merge το pull request, θα μπορέσεις να διαγράψεις το topic branch ώστε να κρατάς τακτοποιημένο το fork σου.
Τώρα, κανονικά, θα μπορούσα να κάνω merge το παρόν pull request. Όμως, για λόγους εκπαίδευσης, θα σε ταλαιπωρήσω λίγο. Θα κλείσω το pull request αντί να το κάνω merge. Στη συνέχεια, θα σου έλεγα να διαγράψεις το fork, ώστε να μπορέσεις κάνεις fork ξανά ένα φρέσκο αντίγραφο. Θα κάνεις πάλι clone τοπικά, διαγράφοντας πρώτα το παλιό clone (ή μετακινώντας το σε άλλο path). Τέλος, θα κάνεις τις ίδιες αλλαγές, όμως αυτή τη φορά σε ένα topic branch, αντί για το master.
— Reply to this email directly or view it on GitHub.
+1 for @petros.
Add member info