Skip to content

FT WD - Berlin Jan. 2020 - Vasiliki & Ciwan - #608

Closed
ciwan-aram wants to merge 1 commit into
ironhack-labs:masterfrom
ciwan-aram:master
Closed

FT WD - Berlin Jan. 2020 - Vasiliki & Ciwan#608
ciwan-aram wants to merge 1 commit into
ironhack-labs:masterfrom
ciwan-aram:master

Conversation

@ciwan-aram

Copy link
Copy Markdown

No description provided.

Comment thread js/index.js
console.log("The driver's name is " + hacker1);
let hacker2 = "Vasiliki";
console.log("The navigator's name is " + hacker2);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice! All good on first iteration :D
As well if you guys want to try other way to put variables in a string you could try the `` quotes where you could write your string like this The driver´s name is ${hacker1} as an example.

Comment thread js/index.js
"Wow, you have both equally long names, " + hacker1.length + " characters!"
);
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All good in the hood guys!
Only comment about this iteration is the same as the comment above regarding the variables in a string :D

Comment thread js/index.js
console.log("Yo, the navigator goes first definitely.");
} else {
console.log("What?! You both have the same name?");
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice job! No more comments

@stale

stale Bot commented Feb 6, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the stale label Feb 6, 2020
@stale

stale Bot commented Feb 8, 2020

Copy link
Copy Markdown

This pull request is closed. Thank you.

@stale stale Bot closed this Feb 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants