search

PieterTack / polycap

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

version test added #50

Closed PieterTack closed 4 years ago

PieterTack commented 4 years ago

not sure how the vasprintf thing works with strcmp. As I understand the first returns an int, not a char...

tschoonj commented 4 years ago

Dit gaat dus idd niet werken. En ge hebt blijkbaar niet gewacht op de CI hier voor te mergen 😉

tschoonj commented 4 years ago

Oh en ge zijt hiet rare dingen aan het doen met uw branches: deze hier is bijvoorbeeld niet naar master gegaan.

PieterTack commented 4 years ago

ja, sorry :( Kheb nog steeds niet gevonden hoe ik op mijn remote client gemakkelijk van een bestaande pull request kan branchen :(

tschoonj commented 4 years ago

Normaal moet ge dat nooit doen: branches van PRs zijn veronderstelt onafhankelijk te zijn van elkaar aangezien ze voor verschilende features of bug fixes bedoeld zijn.

PieterTack commented 4 years ago

True, maar gisteren had ik bijvoorbeeld de pull request gemaakt door in de browser snel al een verandering aan te brengen in configure.ac. Vandaag wou ik daarop verder werken, vanuit mijn remote client. Hoe kan ik dan mijn commits laten terecht komen in diezelfde pull request? Het gaat immers om dezelfde feature of bug fix. Kvermoed dat ik dan ergens moet git push --set-upstream, maar wat zet ik dan als extra parameters?

tschoonj commented 4 years ago

De PR die ge gemaakt hebt via de browser is ook beland op een branch (version1). Om daar lokaal op te kunnen voortwerken moet je doen:

git fetch
git checkout version1 (dit zal al de upstream goed zetten)
... do some commits...
git push

Gebruikt ge trouwens liquidprompt? Ik raad dit sterk aan als ge werkt met git.