PieterTack / polycap

Polycapillary X-ray raytracing
GNU General Public License v3.0
3 stars 3 forks source link

polycap_description_new: profile first argument #23 #32

Closed PieterTack closed 6 years ago

tschoonj commented 6 years ago

Korte nacht precies 😄

tschoonj commented 6 years ago

Btw als je in een commit message body iets schrijft als Closes #23, dan zal het overeenkomstige issue automatisch afgesloten worden na merge van de PR.

Als je dat zou willen doen voor deze PR is het best dat je de commit gaat amenderen:

git commit --amend

Daarna zal vi terug openen en kan je de message aanpassen. Hierdoor wordt wel de commit geschiedenis herschreven dus zal je moeten force pushen naar Github: git push -f

Commit amenderen is iets dat veel gedaan wordt trouwens: buiten de commit message aanpassen kan je ook changes en zelfs ganse files gaan toevoegen aan de laatste commit. Dit is nuttig als je soms een nieuw bestand bent vergeten toe te voegen met git add na te committen en het nogal onnozel is om daarvoor een nieuwe commit te maken...

Het is wel iets gevaarlijker natuurlijk omdat git push -f dan altijd moet gebruikt worden. Doe dit dan ook nooit op de master branch.

PieterTack commented 6 years ago

to be merged