Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34.2k
docs: re-word project messaging#36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Uh oh!
There was an error while loading. Please reload this page.
Conversation
Fishrock123 commented Dec 3, 2014
LGTM, though I think io.js looks nicer as |
kevinohara80 commented Dec 3, 2014
@Fishrock123 done ^^ |
rvagg commented Dec 4, 2014
"io.js" isn't a programatic name so I'm -1 on |
d7e65ff to 185d11cCompareFishrock123 commented Dec 5, 2014
Yeah Rod is correct, I'm just silly. |
kevinohara80 commented Dec 5, 2014
@rvagg I just removed the back ticks. Agree that formatting should be reserved for programatic items. |
indutny commented Dec 10, 2014
@Fishrock123@rvagg are we good with this? |
rvagg commented Dec 10, 2014
sure, the line breaks bother me but that's not a big deal LGTM |
bnoordhuis commented Dec 10, 2014
I think this needs to be rebased anyway before it can be applied. @kevinohara80 If you could fix the line breaks while you are at it, that would be great. |
kevinohara80 commented Dec 10, 2014
No problem. I'll do this today. On Wednesday, Dec 10, 2014 at 7:43 AM, Ben Noordhuis [email protected], wrote: I think this needs to be rebased anyway before it can be applied. @kevinohara80 If you could fix the line breaks while you are at it, that would be great. — |
kevinohara80 commented Dec 10, 2014
@bnoordhuis@rvagg Rebased and removed whitespace to make it a single paragraph. Let me know if there's any other tweaks. |
bnoordhuis commented Dec 10, 2014
Sorry, I didn't notice it's spread out over multiple commits. Can you squash them and make the commit log conform to what is outlined in CONTRIBUTING.md? |
rewords project messaging in README to make the overall project messaging a bit clearer. More discussion to be found in nodejs#24
kevinohara80 commented Dec 10, 2014
@bnoordhuis My bad. Does this look good? |
Rewords project messaging in README to make the overall project messaging a bit clearer. More discussion to be found in #24. PR-URL: #36 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Rod Vagg <[email protected]>
bnoordhuis commented Dec 10, 2014
Looks great, Kevin. Landed in 6f1d0a8, thanks! |
The previous paragraph was good but a little unclear. This fixes that.