Skip to content

Louis Li - JavaScript-II - #300

Open
louislee7 wants to merge 7 commits into
bloominstituteoftechnology:masterfrom
louislee7:master
Open

Louis Li - JavaScript-II #300
louislee7 wants to merge 7 commits into
bloominstituteoftechnology:masterfrom
louislee7:master

Conversation

@louislee7

Copy link
Copy Markdown

@kiadorii

@kiachoi kiachoi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✔️ What Went Well:

  • Callbacks
  • Array Methods
  • Closures
  • Arrow function syntax
  • Clean commit messages and code

🎗 Recommendation(s):

  • When you feel more comfortable, start using let or const in replacement to var.

Other:

  • 3/3

Comment thread assignments/callbacks.js Outdated

function firstItem(arr, cb) {
// firstItem passes the first item of the given array to the callback function.
cb(arr[0]);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to return this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants