-
Notifications
You must be signed in to change notification settings - Fork 29
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
Guides may require a proofread? #20
Comments
Sure, go ahead. Thanks! On Tue, Aug 13, 2013 at 6:55 AM, Crispen Smith [email protected]
|
I'm working on it now, it's a fairly thorough proof read, so I'll send you a note when I've complete to see if you have any thoughts before I submit a pull. In working on it I realized how much I appreciate the details that you've put into this. I wasted a day on another framework because this simple line wasn't included in the docs:
(I copied after I proposed the edit to this block). I came to node having written front-end javascript for several years so my conception was that the whole thing would be reloaded on every refresh, putting a warning about this right in the docs is a brilliant move. |
As per issue compoundjs#20, proofreading and general clean up to this manual for the purpose of readability.
I've made a first pass over at: https://github.com/crispen-smith/guides/blob/master/basics/crash-course.md also, the intro blurb for the crash course at: https://github.com/crispen-smith/guides/blob/master/README.md |
Thanks man, I reviewed your work and I think you doing it great. Do not hesistate add your own thoughts, because it's very important to see framework from the different points. My own vision: everything is obvious, but of course for newcomer it isn't true, that's why it's good if newcomer will contrubute and clarify some unclear parts. Good luck! |
That's my thinking as well. It's difficult for a code author to write docs Writing docs that are intended for someone with no understanding of a I do have a couple of questions about specific sections, but those are On Tue, Aug 13, 2013 at 10:53 AM, Anatoliy Chakkaev <
|
I'm super frustrated with myself for not getting back onto this sooner... August turned into a really deep development cycle where I hardly came up for air. I'm hoping to start touching this again in the next day or two. I'd sent you an email to the address you gave me, about mid-month, did you see it or did I get spamfiltered? |
Sorry man, I haven't noticed it in my inbox, because it was busy days, as On Fri, Aug 30, 2013 at 3:13 AM, Crispen Smith [email protected]:
Thanks, |
No worries. I'd been more worried about it being eaten by your spam filters than anything else. I hope the London move is driven by good things. |
Heya, Sorry I've been so quiet on my end the last couple of months. Went through some fairly crippling life events. However, all seems to be sorted now. Is there still value in my reviewing docs? |
Yep, this is always actual task. Also if you have time and want to try our On Wed, Nov 6, 2013 at 2:24 AM, Crispen Smith [email protected]:
Thanks, |
I'm definitely up for working on the docs for CompoundJS... I think it's a As far as committing any sort of time to another project, I'm interested On Thu, Nov 7, 2013 at 8:34 AM, Anatoliy Chakkaev
|
Nko is 48hour challenge. Details by link. On Fri, Nov 8, 2013 at 12:42 AM, Crispen Smith [email protected]
|
Hi!
I'm sorry if this is out of line but I seem to have noticed a pattern with the basics/crash-course.md file where phrases are missing the word "the", for example "Let's start with routes, because this is first place in the compound stack", my feeling is that this might read better as "Let's start with routes, because this is the first place in the compound stack".
From what I've read so far the compoundjs may soon become my new goto back-end framework, I'd be happy to do a pull request with some proofreading done on the crash-course.md file.
The text was updated successfully, but these errors were encountered: