Please accept this documentation update

[krosenvold]

I have signed the CLA ;)

[lukeis]

fixed with d835b9b

(annoying to not see the sync happen and thus github sees two commits here)

Also do you think we should follow the maven git convention and 'alter' the commit message as I did?

[lukeis]

Looks like I changed the Author too, so i guess I shouldn't do that :)

[krosenvold]

I think we specifically want to avoid changing the author. I suggested
using --signoff --amend which is "simple".

Kristian

2013/1/15 Luke Inman-Semerau [email protected]

Looks like I changed the Author too, so i guess I shouldn't do that :)

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-12256869.

[lukeis]

Yep, makes sense. no beer for me :)

On Tue, Jan 15, 2013 at 12:36 AM, Kristian Rosenvold <
[email protected]> wrote:

I think we specifically want to avoid changing the author. I suggested
using --signoff --amend which is "simple".

Kristian

2013/1/15 Luke Inman-Semerau [email protected]

Looks like I changed the Author too, so i guess I shouldn't do that :)

—
Reply to this email directly or view it on GitHub<
https://github.com/SeleniumHQ/selenium/pull/1#issuecomment-12256869>.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-12257812.

[detro]

So, this is totally bidirectional :)

[lukeis]

Ivan, not how we intend to use it (as in, please make sure you read
Kristian's emails on the dev list). No direct pushes to github (no using
the github gui to 'merge' a pull request).

On Tue, Jan 15, 2013 at 12:40 AM, Ivan De Marino
[email protected]:

So, this is totally bidirectional :)

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-12257902.