search

CalZoneVUB / Documents

Bevat de te opleveren documenten, alsook user manuals etc. (Geen documentatie van de code.)
0 stars 2 forks source link

Branch Vergadering 5 #2

Closed MPieter closed 11 years ago

MPieter commented 11 years ago

Ik heb een branche aangemaakt voor vergadering 5. Moet ik die mergen of de verantwoordelijke voor dit document ?

gaethofs commented 11 years ago

Ik denk dat we dit gewoon beter nog als pdf publiceren en donderdag na de vergadering de workflow en de werking met de branches hiervoor vastleggen. Niet?

Timbo925 commented 11 years ago

Kunde geen pull request doen? Dan kan ik even doen alsof ik de verantwoordelijke ben voor het document en de commit hiervan doen.

Donderdag effectieve workflow vastleggen lijk me inderdaad het beste.

Timbo925 commented 11 years ago

Heb het eens geprobeerd en pull request maken lukt dus makkelijk op de site. Eigenaar van documenten can dan een review doen en beslissen om te mergen of niet.

MPieter commented 11 years ago

Workflow vastleggen donderdag is inderdaad het beste -> in het verslag van vergadering 5 staat al dat er volgende donderdag workshop over Github door Christophe wordt gegeven.

Dit komt ook nog in de agenda.

MPieter commented 11 years ago

Als ik het goed begrijp heeft Tim die pull request nie zelf uitgevoerd? Want ik heb hier wel een mail over gekregen.

Dit werkt dan wel enkel goed als de eigenaar van de documenten goed is aangeduid ?

Timbo925 commented 11 years ago

Ik heb de pull request aangemaakt en dan zelf gemerged. Maar omdat iedereen eigenaar is krijgt iedereen waarschijnlijk een mail dat hij een pull request heeft gekregen over code die aan hem besteed is.

Werk beetje hetzelfde als issues die je aan iemand assigned