-
Notifications
You must be signed in to change notification settings - Fork 313
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
Streams adventure, not #137
Comments
It's a great set of exercises introducing people to Node and to streams generally, though. |
This lesson actually only teaches how to use |
Hi @famousgarkin do you have suggestions for the topics? |
I'm not OP but I agree with the points he's making. As someone starting off in Node I'm confused with the different forms Streams take in the API, which is why I was drawn to this workshop. But it really didn't answer any of these questions. |
@kalinchernev for me, I would suggest working more within Node's core functionality. Both the Learnyounode & this workshop have the student install through2 to achieve a result that is much easier accomplished within the javascript language. Aside from that, the exercises that I've completed all do very little to "teach", settling instead for coming just short of providing the answers to their respective problems. I would have preferred the workshop to actually play into the "adventure" theme and challenge the student with more demanding problems. EDIT: Ok, "Beep Boop" was simple, but the follow up "Meet Pipe" could have been made better by returning an actual file to the stdout with adventure-flavoured content(?). Also perhaps by not actually providing the solution in the problem. |
Not complaining, nor whining, but the adventure teaches close to nothing about Node.js streams. I'd sum it up as couple examples of Node.js stream-related packages. The tough part was going through quirks of assignment descriptions and packages usage. Expected something along the lines of using and implementing bare Node.js streams. So just a thing to think about, if this is a good direction overall.
The text was updated successfully, but these errors were encountered: