add title to things about sioux_tech_radar HOT 7 CLOSED

Voorstel: het woord Thing altijd met een hoofdletter typen, anders raak ik in de war :-)

ps. had je mn pull request gezien?

from sioux_tech_radar.

ja, ik zag het. Je heb het ook al gepulled zag ik.

Was dat een pull request van een branch in dit project naar de master van
dit project?

On Thu, Apr 4, 2013 at 8:16 AM, Egbert Teeselink
[email protected]:

Voorstel: het woord Thing altijd met een hoofdletter typen, anders raak ik
in de war :-)

ps. had je mn pull request gezien?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/16#issuecomment-15881538
.

from sioux_tech_radar.

Huh? Het was niet de bedoeling om die pull request te pullen. Damn, wat is git toch ingewikkeld soms.

Het was inderdaad een request van een branch ("API") naar master. Had je van die pull request een notification mail gekregen? A.k.a. is dit de juiste manier om "hee review dit eens!" commits langs elkaar te laten sturen?

from sioux_tech_radar.

dus feature branches maken en die door elkaar laten reviewen via pull
requests. Klinkt als een process dat ik wil uitproberen.
ik ben voor.
Volgende taak die ik pak is het toevoegen van title in Thing, die zal ik in
een branch stoppen.

2013/4/4 Egbert Teeselink [email protected]

Huh? Het was niet de bedoeling om die pull request te pullen. Damn, wat is
git toch ingewikkeld soms.

Het was inderdaad een request van een branch ("API") naar master. Had je
van die pull request een notification mail gekregen? A.k.a. is dit de
juiste manier om "hee review dit eens!" commits langs elkaar te laten
sturen?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/16#issuecomment-15883185
.

from sioux_tech_radar.

Akkoord.

Ik stel voor dat we het alleen maar doen als we willen
dat de ander ff reviewt. als je't wel gelooft, dan kun je de boel wat
mij betreft direct naar master doen.

Nog iets anders: taal. de UI taal
is NL. Code is Engels. Commit messages en issues zijn een bende. Zullen
we daar ook NLs doen?

On 2013-04-04 09:45, Ronald Crooy wrote:

dus
feature branches maken en die door elkaar laten reviewen via pull

requests. Klinkt als een process dat ik wil uitproberen.
ik ben voor.

Volgende taak die ik pak is het toevoegen van title in Thing, die zal
ik in
een branch stoppen.

2013/4/4 Egbert Teeselink
[email protected]

Huh? Het was niet de bedoeling om die
pull request te pullen. Damn, wat is
git toch ingewikkeld soms.

Het was inderdaad een request van een branch ("API") naar master.
Had je
van die pull request een notification mail gekregen? A.k.a.
is dit de
juiste manier om "hee review dit eens!" commits langs
elkaar te laten
sturen?

Reply to this email
directly or view it on
GitHubhttps://github.com//issues/16#issuecomment-15883185

.

Reply to this email directly or view it on
GitHub [1].

Links:

[1]
#16 (comment)

from sioux_tech_radar.

voortaan alles NL. behalve dan de code en de comments. ... of ook de
commetns?

2013/4/4 Egbert Teeselink [email protected]

Akkoord.

Ik stel voor dat we het alleen maar doen als we willen
dat de ander ff reviewt. als je't wel gelooft, dan kun je de boel wat
mij betreft direct naar master doen.

Nog iets anders: taal. de UI taal
is NL. Code is Engels. Commit messages en issues zijn een bende. Zullen
we daar ook NLs doen?

On 2013-04-04 09:45, Ronald Crooy wrote:

dus
feature branches maken en die door elkaar laten reviewen via pull

requests. Klinkt als een process dat ik wil uitproberen.
ik ben voor.

Volgende taak die ik pak is het toevoegen van title in Thing, die zal
ik in
een branch stoppen.

2013/4/4 Egbert Teeselink
[email protected]

Huh? Het was niet de bedoeling om die
pull request te pullen. Damn, wat is
git toch ingewikkeld soms.

Het was inderdaad een request van een branch ("API") naar master.
Had je
van die pull request een notification mail gekregen? A.k.a.
is dit de
juiste manier om "hee review dit eens!" commits langs
elkaar te laten
sturen?

Reply to this email
directly or view it on
GitHub<
#16 (comment)

.

Reply to this email directly or view it on
GitHub [1].

Links:

[1]

#16 (comment)

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/16#issuecomment-15886384
.

from sioux_tech_radar.

Nee niet de comments, want dan ontploft mijn hoofd :-)

Ronald Crooy [email protected] wrote:

voortaan alles NL. behalve dan de code en de comments. ... of ook de
commetns?

2013/4/4 Egbert Teeselink [email protected]

Akkoord.

Ik stel voor dat we het alleen maar doen als we willen
dat de ander ff reviewt. als je't wel gelooft, dan kun je de boel wat
mij betreft direct naar master doen.

Nog iets anders: taal. de UI taal
is NL. Code is Engels. Commit messages en issues zijn een bende. Zullen
we daar ook NLs doen?

On 2013-04-04 09:45, Ronald Crooy wrote:

dus
feature branches maken en die door elkaar laten reviewen via pull

requests. Klinkt als een process dat ik wil uitproberen.
ik ben voor.

Volgende taak die ik pak is het toevoegen van title in Thing, die zal
ik in
een branch stoppen.

2013/4/4 Egbert Teeselink
[email protected]

Huh? Het was niet de bedoeling om die
pull request te pullen. Damn, wat is
git toch ingewikkeld soms.

Het was inderdaad een request van een branch ("API") naar master.
Had je
van die pull request een notification mail gekregen? A.k.a.
is dit de
juiste manier om "hee review dit eens!" commits langs
elkaar te laten
sturen?

Reply to this email
directly or view it on
GitHub<
#16 (comment)

.

Reply to this email directly or view it on
GitHub [1].

Links:

[1]

#16 (comment)

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/16#issuecomment-15886384
.

---

Reply to this email directly or view it on GitHub:
#16 (comment)

from sioux_tech_radar.