Skip to content

Nedim Omerovic - JavaScript II - #418

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

Nedim Omerovic - JavaScript II#418
Civoremo wants to merge 5 commits into
bloominstituteoftechnology:masterfrom
Civoremo:master

Conversation

@Civoremo

Copy link
Copy Markdown

No description provided.

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

Not much to suggest here, Great job sir.

// The donations need to be tallied up and reported for tax purposes. Add up all the donations into a ticketPriceTotal array and log the result
let ticketPriceTotal = [];
const totalDonation =
runners.reduce(function(money, arr){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Try some more semantic parameters. The second one, for example is the element of the array, not the array. Something like "runner" or "racer" might be reasonable.

Comment thread assignments/closure.js
const favDay = "31st";
console.log(`Halloween is on ${month}, ${favDay}.`);

function today() {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is great

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