Připravit git hooks na kvalitní commit message

manGoweb / Skeleton

:grapes: Skeleton for Nette 2.2 and onward

Tajná zpráva :P

Other

1 stars 0 forks source link

Připravit git hooks na kvalitní commit message #18

Open Mikulas opened 10 years ago

Mikulas commented 10 years ago

Error první linka nesmí přetýct
warn pokud to nezačíná na add, fix, remove, nebo update
Erorr pokud mezi první linkou a třetí neni newline
warn pokud tam není issue id
- Error pokud je tam issue id a není v commit body a není formát [Update #n] [Close #n]

hrach commented 10 years ago

warn pokud to nezačíná na add, fix, remove, nebo update

dost blbost, kvalitni commit message vubec nezacina takto, kvalitni commit message jasne vytycuje predmet, ceho se to tyka. po te je treba dvojtecka a pote muze byt check na toto slovo.

warn pokud tam není issue id

Znam jenom jednu firmu, ktera ma na vsechno issue, a to je mozilla. Naprosto neaplikovatlene pro agilni vyvoj.

Error pokud je tam issue id a není v commit body a není formát [Update #n] [Close #n]

osobne pouzivam [closes #n] takze vzor vyse mi zasadne vadi :P

Mikulas commented 10 years ago

Pointa toho je to sjednotit, klidně to může být to co navrhuješ ty. Všimni si že většina toho je jenom warning, rozhodně každý commit issue mít nemusí.

Hlavně je to takový první drápeček na git hooks, časem bych vás rád přesvědčil že není úplně špatné mít na pre-commit i testy atp.

hrach commented 10 years ago

Pokud mam warning u 90 % commitu, pak ho zacnu ignorovat a nema smysl tehdy, kdyz je ho treba. Pre-commit testy - zni to dobre, ale ja bych si to nedal :)