Skip to content

Improve text in README.md#9

Open
jankeromnes wants to merge 1 commit into
steeve:masterfrom
jankeromnes:patch-1
Open

Improve text in README.md#9
jankeromnes wants to merge 1 commit into
steeve:masterfrom
jankeromnes:patch-1

Conversation

@jankeromnes

Copy link
Copy Markdown

Thank you for doing this! I've been toying with the idea for a while, so I'm extremely happy that this now exists.

There were a few minor typos and confusing expressions in the README, so I fixed them.

@pyrou

pyrou commented Mar 31, 2015

Copy link
Copy Markdown

verisonnement => versionnement

@jankeromnes

Copy link
Copy Markdown
Author

Thanks for the quick catch @pyrou! Fixed, along with a few "celà".

@emerick42

Copy link
Copy Markdown
Contributor

You should rebase your PR, as it's a duplicate of #1 (which was merged recently).

@steeve

steeve commented Mar 31, 2015

Copy link
Copy Markdown
Owner

i feel bad for all the typos, but i was writing with my qwerty keyboard falling asleep at 3am haha

thanks guys, this is what open source is about :)

@jankeromnes

Copy link
Copy Markdown
Author

@steeve Rebased on top of the new changes. Includes the same fixes as pull request #16, plus a few others.

@EHadoux

EHadoux commented Apr 2, 2015

Copy link
Copy Markdown

Please see #23 and #24 for more.

@jankeromnes

Copy link
Copy Markdown
Author

My these get out-dated quickly. I'll give up and let the more eager take over :)

@jankeromnes jankeromnes closed this Apr 2, 2015
@EHadoux

EHadoux commented Apr 2, 2015

Copy link
Copy Markdown

No, no, I guess all those PR (this one #9, #16, #23 and #24) do not have the same fixes. They all need to be merged.

@jankeromnes

Copy link
Copy Markdown
Author

@EHadoux My pull request is already stale because others have been merged first, and I'm too lazy to rebase :) feel free to steal my changes and rebase them into your commits though!

@emerick42

Copy link
Copy Markdown
Contributor

The part about the rewrite using french words would have been interesting (I'm not sure it has already been changed elsewhere).

@jankeromnes

Copy link
Copy Markdown
Author

Ok, I'll re-open this then. But please feel free to use my changes elsewhere if this doesn't get merged, I don't really care about attribution :)

@jankeromnes

Copy link
Copy Markdown
Author

Rebased on top of new HEAD.

@emerick42

Copy link
Copy Markdown
Contributor

I think we should try to merge it once @steeve has merged other PRs about modifying the readme (or even before, it doesn't impact that much).

Comment thread README.md

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Krzysztof Tadeusz Pieton live in 139 Church Hill Tullamore Offaly Ireland

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants