Szerintem egyértelmű, hogy amikor az ember patchet ír, akkor alkalmazkodik az adott projekt igényeihez.
- úgy indentálok, ahogy a projekt
- a patchet elsősorban a trunk-ra kell megírni (hiszen az lesz a jövő, a core fejlesztők is arra koncentrálnak) még akkor is ha nekem egy release-re kell a fix (akkor kell még egy patchet írni a branchre)
- dokumentálni, tesztet írni szintén a projektben elvártak szerint (és nem azért mert az embernek erre van ingere)
- nem levlistára tölti az ember a patchet, hanem issue trackerben esetet nyit neki és feltölti oda. Így nem veszik el/felejtődik el és más is fel tudja venni a fonalat akár évekkel később is.
Patch íróként a projekt gazdák fejével is kell gondolkozni. Különösen ha nem egy bug fixről van szó, hanem egy új feature-ről. Te szeretnéd ezt az új funkciót, de a projekt fejlesztőinek is megvannak a saját tervei, nem feltétlen arra várnak, hogy végre valaki igényét lefejleszthessék. Szóval úgy kell tálalni az igényt és a patchet, hogy a fogadó azt a lehető legkisebb megerőltetéssel tudja alkalmazni.