Skip to content

Commit a4ffc85

Browse files
committed
Address the issues from the pull request comments
1 parent 7659c5a commit a4ffc85

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

Sprint-1/1-key-exercises/1-count.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,5 @@ count = count + 1;
55
// Line 1 is a variable declaration, creating the count variable with an initial value of 0
66
// Describe what line 3 is doing, in particular focus on what = is doing
77

8-
// Line 3 is a reassignment of the variable `count` to its current value plus 1
8+
// Line 3 is a reassignment of the variable `count` to its current value plus 1.
9+
// In other words, it increment by 1

Sprint-1/3-mandatory-interpret/1-percentage-change.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ console.log(`The percentage change is ${percentageChange}`);
1212
// Read the code and then answer the questions below
1313

1414
// a) How many function calls are there in this file? Write down all the lines where a function call is made
15-
// Line 4, 5
15+
// Line 4, 5, 10
1616

1717
// b) Run the code and identify the line where the error is coming from - why is this error occurring? How can you fix this problem?
1818
// It's missing a comma between the arguments at line 5 in replaceAll() method
@@ -24,5 +24,5 @@ console.log(`The percentage change is ${percentageChange}`);
2424
// Line 1, 2
2525

2626
// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
27-
// Its purpose is to replace a substring with another string, in this case, replaces a command (",") with an empty string ("")
28-
// In order words, remove the comma (",") in the string
27+
// Its purpose is to replace a substring with another string and then convert it into a number, in this case, replaces a command (",") with an empty string ("")
28+
// In other words, remove the comma (",") in the string and then convert it into a number

Sprint-1/4-stretch-explore/chrome.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,4 +26,4 @@ It displays an alert with a textfield where you can enter some text, e.g. John.
2626

2727
What is the return value of `prompt`?
2828

29-
The string returned from the textfield what the user has entered otherwise null.
29+
The string returned from the textfield what the user has entered otherwise null if the user left it blank.

0 commit comments

Comments
 (0)