Skip to content

done - #667

Closed
valmgisbert wants to merge 4 commits into
ironhack-labs:masterfrom
valmgisbert:master
Closed

done#667
valmgisbert wants to merge 4 commits into
ironhack-labs:masterfrom
valmgisbert:master

Conversation

@valmgisbert

Copy link
Copy Markdown

I'll dO THE BONUS 2 LATER I AM T I R ED

Comment thread js/index.js
// Iteration 1: Names and Input
//it 1
var hacker1 = 'Jason';
console.log(`"The driver's name is ${hacker1}"`);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you use the string interpolation, you don't need the quote inside, but you solve that later in the exercises, so good :)

Comment thread js/index.js
}

//it 3
for (i = 0; i <= hacker1.length - 1; i++) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you need to assign i to a variable, var i = 0

@tawebbcn tawebbcn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Really good job, all competeted !
Congrats 👍

@tawebbcn tawebbcn closed this Jan 14, 2020
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