Skip to content

Update README file - Extend explanations, fix formatting mistakes and expand examples #2938

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

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Prev Previous commit
Next Next commit
Update Readme during review
  • Loading branch information
bstashchuk committed Apr 12, 2024
commit 357f3cf7e12442fa8f1b763966f8dac6c90178d8
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@ Other Style Guides
```

<a name="arrays--mapping"></a>
- [4.6](#arrays--mapping) Use [`Array.from`](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/from) instead of spread `...` for mapping over iterables like Sets, Maps or NodeLists, because it avoids creating an intermediate array.
- [4.6](#arrays--mapping) Use [`Array.from`](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/from) instead of spread `...` for mapping over iterables, because it avoids creating an intermediate array.

```javascript
// bad
Expand Down Expand Up @@ -710,7 +710,7 @@ Other Style Guides

// good
if (currentUser) {
let test = () => {
const test = () => {
console.log('Yup.');
};
}
Expand Down