GitHub Messages in der Liste

[delphiN]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am 12.11.2014 12:53, schrieb Tim Niemeyer:
> http://justinhileman.info/article/changing-history/ 
> https://guides.github.com/introduction/flow/index.html

Ok, Ich lerne, oder führe mir aufs Neue vor Augen:
- ---------------------------

* So wenig Commits machen wie möglich, so viele wie nötig.

* Jeder Commit sollte ein einzelnes Thema betreffen. Zusammenhängende
Commits am besten schon vor dem publishen zusammenführen. (Use "gitx"
or "git add -p" to stage partial-file changesets.).

* Nur getesteten Code publishen.

* Vor jedem Pull-Request ein rebase oder ein merge mit master durchführen.

* Je schöner und übersichtlicher Pull-Request sind desto einfacher ist
es sie zu reviewen.

* Es gibt noch einige GitHub Anfänger. Durch eine ganz oder garnicht
Mentalität schließt man Anfänger und andersdenkende komplett aus und
schadet dem ganzen Projekt. Toleranz!


Und du lernst (hoffentlich):
- ----------------------------

* "GitHub has an amazing code review system called Pull Requests":
Nutze Pull-Request um Änderungen zum Review zu stellen (keiner will
mehr olle Patches per mail)

* "Pull Requests preserve a record of the historical changes to your
code": Nutze GitHub-Merge um Pull-Requests als ganzes in den master zu
mergen. Der dadurch entstehende Merge-commit macht klar, dasss es sich
hier um ein Patch-Set und nicht einzelne Patches handelt.

* Bitte auch keine einzelnen Commits aus einem Pull-Request mergen es
sei denn der Pull-Request wird danach tatsächlich verworfen und das
ist mit dem Contributer so abgesprochen. Das schafft "Chaos" und mach
dem Contributer das Leben unötig schwer.

* Wenn du Anmerkungen zu einem speziellen Commit hast nenn diesen auch
beim Namen (Hash). Commit-spezifiche Anmerkungen gehören als Kommentar
zu dem Commit. Ganze Pull-Request nicht wegen einzelner Commits
verteufeln!

* Auch "unschöne" Pull-Request (Patchsets) kann man als Ganzes
inhaltlich reviewen und Rückmeldung geben. Mann muss sich in GitHub
nicht von Patch zu patch hangeln.

* Es gibt noch viele Git Anfänger. Durch eine ganz oder garnicht
Mentalität schließt man Anfänger und andersdenkende komplett aus und
schadet dem ganzen Projekt. Toleranz!

* Pull-Request sollten nur vom Contributer selbst oder durch ein Merge
geschlossen werden.

* Wir entwickeln nicht der schönen Git-Historie wegen sondern wegen
der Software. Im Zweifelsfall auch unschöne "Patch-Sets" mergen wenn
der Code passt. Den Contributer auf Verbesserungen hinzuweisen schadet
allerdings nicht.


delphiN

- -- 
freifunk at wunschik.net
delphiN at jabber.ccc.de
https://github.com/FreifunkFranken


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)

iQIcBAEBAgAGBQJUY2TyAAoJEGuH2dOBPapCiUYP/RP/zAARsWFIB9nP5quBCOaK
LqczNk/Ph2GdDhJW9+etstOmXzMPwmImjL3RuEp/0mw2Iv3wJmquxE8I0CkuZBV7
W/XwqJ+wWHHnoBhGpMwXmRc+RDFBv1xIhig06oUAAJcrmjP26Is8XkTuInxLMQ8w
zbPrt4/D+3yI74vSd/5/Vhhx2XmMFqXXT7dnxhtJ86YQsGwvO6RufHC+3lgZ2fSa
96eEd549PZmLHkMzzRGJpSKvFjXuMI7G4t2kkg35qPN5agalfyfEm7WRgVDFjJSr
2rwLQcPMHRM/aYR+7BJ9GGHVr4RwjG7vcmCH3csTXZee4SlPcMffH+ySx9H9dVOe
eVKxW5e3jqwWBJ3VdR9+YAB1TaME8F4HaRkKHtDLna8AUucRODUzc9y97cIPDyIq
YSPiT1bZbrQLV3qX1uApMDAPeXBp/Vi2YGoVEZ8+6gnFkzarK4+qJaZVuWNikPS1
JRC1RsQ3KkHLCHKCp9SHKeq6dDYBHWzpP+vlEchOuLqS4N4KuY/MPYHgUaXxzBFA
jkTyS3d6RPUu7NDCxDWM08mz1988UvD0v0Ld1vouohogBfonhn0Kwk3+5edbMFgQ
yp3N+DbVV/zLF23Y3tfBHFcbgsW+BopNo27klWyPTtKs8AUDA+dg1PSSvKiNoHWB
th/UphqXU+7z1hSn4FGM
=APAQ
-----END PGP SIGNATURE-----