Bringing fork up to date #1
Merged
Conversation
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
* translate kata pengantar / introduction * translate variabel * typo * function 1/2 * 1/2 replace * translate ### 😂 * translate Function section in Bahasa Indonesia is done * 1/2 SOLID translation to Bahasa Indonesia * SOLID translation to Bahasa Indonesia is Done * translating Error Handling to bahasa indonesia is done * well done, translated to Bahasa Indonesia * review 1, check typo * review 2 * just link
Capital letters in const, remove default data in class Car
Capital letters in const, remove default data in class Car
The proposed "good" way of coding was actually changing a property's name in the developers object. While one can argue that in this case they could be the same, the point of the example was to show that you can still remove duplicated code that require *different* things.
Fixed duplicate code
Remove space before colon in object.
Object style consistency
Update README.md
'data' is never reassigned. Use 'const' instead.
…turing Replace map method with object destructuring
There's -> There are
minor grammar edit
Update README.md
It doesn't need to be a global constant. JavaScript constants are block-scoped.
Fix "Use searchable names" section
Fix "Use searchable names" section
Change complex reduce code sample to clearer map reduce.
…iting or conditionals". Did not adhere to "Don't add unneeded context".
Change argument name of "Use default arguments instead of short circu…
…bstraction* Like said in https://github.com/ryanmcdermott/clean-code-javascript#function-callers-and-callees-should-be-close, the code is read from top to bottom, so I'd prefer the definition of the *function parseBetterJSAlternative* above the functions called within.
Changing order in example of *Functions should only be one level …*
Hey there. As a Software Engineer who happens to be Nigerian, I think there are better ways of explaining a side effect than perpetuating stereotypes about a specific group of people. So I just made a small fix to make it more appropriate.
Use `console.error` at least as `console.log` essentially turns a thrown error into a muted console log.
In the "good" `Object.assign` example, the initial new object is redundant as the defaults are also specified in a new object. This is a bit pedantic, and doesn't change the semantics of the example, just removes an unnecessary object creation.
The original phrase "As stated in the Gang of Four..." is semantically incorrect. - The "Gang of Four" is a group of authors: Erich Gamma, Richard Helm, Ralph Johnson and John Vlissides. - They wrote a famous book: Design Patterns. - Here, Ryan McDermott is referring to the software-design principle "you should prefer composition over inheritance." - This principle was stated in the book, Design Patterns. - It was not stated in the Gang of Four -- the authors themselves.
per preceding rule "ES6 constants when variable values do not change"
In the section "Functions should do one thing" this change uses the Array.filter method to get the active clients, then emailing those with the forEach loop.
The save method isn't returning this, so the car variable is equal to undefined instead of a Car object. BTW, Thanks alot for this guide.
…witch to "ES<year>/ES<edition>" when mentioning ECMAScript versions.
Something looked off about this so I did a quick check in node. You must have been distracted at the time of writing :)
We need a better explanation for when to prefer `const` over var/let, as this has confused many readers.
Add italian translation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.