This repository was archived by the owner on May 14, 2024. It is now read-only.
Adds VSCode workspace settings + Eloquent JavaScript - #5
Merged
Conversation
mkruijt
approved these changes
May 2, 2018
mkruijt
left a comment
Member
There was a problem hiding this comment.
thank you jim looks great, I will take another look at the readings and some some more optional references
| - Implement both feedback from Trello and Github. | ||
| - Check on one of your fellow students code and issues and see if her or she implemented their feedback correctly. If there are some things that can be improved make an issue suggesting further improvements. If you think that the feedback has been implemented correctly create a issue saying something like: "nice work you can clear your issues". | ||
|
|
||
| ## Step 2: Reorganize your GitHub |
Member
There was a problem hiding this comment.
hi Jim why did you move this out?
Collaborator
Author
There was a problem hiding this comment.
Hi Maartje, I moved this out because I think checking the structure should be part of the homework review, at least, that's how I have done it. In the MAKEME there are specific instructions on how to set up the repo and we actually created the hyf-javascript1 repo on GitHub during the lecture. And of course, as of JS2 we expects student to fork an existing repo.
| ## Step 0: Make a small dance | ||
| Give yourself (or your neighbor) a little tap on the shoulder, you've made it to JS2! :muscle: | ||
|
|
||
| ## Step 0: review |
|
|
||
| Go through the topics of week 2-4. | ||
| Review through the topics of JavaScript1: | ||
|
|
| These chapters from _Eloquent JavaScript_ give in-depth explanations of the topics that will be discussed during the lecture. Highly recommended (if time permits). | ||
|
|
||
| - Chapter 13: [JavaScript and the Browser](http://eloquentjavascript.net/13_browser.html) | ||
|
|
| _You can skip the following sections:_ | ||
|
|
||
| - Moving through the tree | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I have replaced the readings from A Smarter Way with readings from Eloquent JavaScript 3rd Edition.
Reason: