Skip to content

Almost finished - #219

Open
Csillag61 wants to merge 5 commits into
bloominstituteoftechnology:masterfrom
Csillag61:master
Open

Almost finished#219
Csillag61 wants to merge 5 commits into
bloominstituteoftechnology:masterfrom
Csillag61:master

Conversation

@Csillag61

Copy link
Copy Markdown

No description provided.

@gooseandmegander

Copy link
Copy Markdown

Thanks for the PR.

@gooseandmegander gooseandmegander 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.

Thanks, Csilla. I've left some comments for you to review.

Comment thread assignments/callbacks.js
});


function multiplyNums(x, y, cb) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Don't forget to invoke the function.

Comment thread assignments/closure.js
// Example usage: const newCounter = counter();
// newCounter(); // 1
// newCounter(); // 2
let count=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.

I know you are still reading and researching before finishing this one. If you need more resources or anything, let me know.

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