Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

London Class 7 - Aisha Khan - JavaScript-Core-1-Week 1 - #38

Closed
aishak05 wants to merge 5 commits into
CodeYourFuture:masterfrom
aishak05:master
Closed

London Class 7 - Aisha Khan - JavaScript-Core-1-Week 1#38
aishak05 wants to merge 5 commits into
CodeYourFuture:masterfrom
aishak05:master

Conversation

@aishak05

Copy link
Copy Markdown

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Comment thread exercises/C-variables/exercise.js Outdated
// Start by creating a variable `greeting`


var greeting = "Hello World"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Perhaps you could add a (;) at the end of this variable.

console.log(message);
var name = "Aisha";
var nameLength = name.length;
var message = "My name is " + name + " and the length of name is " +nameLength +" characters long."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Maybe we could do with a pronoun just before name on " and the length of name is " and a space before nameLength.

console.log(roughPercentageStudents);

const percentageMentors = numberOfMentors / sum *100;
const roughPercentageMentors = Math.round(percentageMentors);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Lovely labelling of variables

@github-actions

Copy link
Copy Markdown

Your coursework submission has been closed because nobody has interacted with it in 30 days. You are welcome to re-open it to get more feedback.

@github-actions github-actions Bot added the Stale label May 13, 2021
@github-actions github-actions Bot closed this May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants