Skip to content

[BRL-FT-01.2020_VICTORIA_BLATKOWSKA&ELSA FILLIPIDOU] #634

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

[BRL-FT-01.2020_VICTORIA_BLATKOWSKA&ELSA FILLIPIDOU] #634
VictoriaBlat wants to merge 1 commit into
ironhack-labs:masterfrom
VictoriaBlat:master

Conversation

@VictoriaBlat
Copy link
Copy Markdown

First Lab

Comment thread js/index.js

let hacker1 = "Victoria";

console.log(`"The driver's name is ${hacker1}."`);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

In this case and in the hacker2 version the double quotes are unnecessary :) the backticks are already giving you a string. :)

Comment thread js/index.js
let arrayCounter = splitted.length;

console.log(arrayCounter);
console.log("'et' appears " + loremIpsum.match(/et/g).length + " times.");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You didn't need to use any Regex , but nicely done!

Copy link
Copy Markdown

@itstheandre itstheandre left a comment

Choose a reason for hiding this comment

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

Great job!!

Keep up the good work :)

@ElsaFil
Copy link
Copy Markdown

ElsaFil commented Jan 8, 2020

@itstheandre thanks for the review! 🤓

@stale
Copy link
Copy Markdown

stale Bot commented Feb 7, 2020

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 7, 2020
@stale
Copy link
Copy Markdown

stale Bot commented Feb 9, 2020

This pull request is closed. Thank you.

@stale stale Bot closed this Feb 9, 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.

3 participants