Manchester | 25-ITP-May | Fithi Teklom | Sprint 3 | 3-stretch #868
Conversation
Fithi-Teklom
added
the
Needs Review
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
1 similar comment
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
Fithi-Teklom
changed the title
Fithi-Teklom
changed the title
cjyuan
left a comment
cjyuan
left a comment
There was a problem hiding this comment.
Branch is not clean (no wonder the code looks familiar)
| // Sum must be greater than 16 | ||
| const sum = numStr | ||
| .split("") | ||
| .map(Number) |
There was a problem hiding this comment.
Can you explain the meaning of this expression, .map(Number)?
| const sum = numStr | ||
| .split("") | ||
| .map(Number) | ||
| .reduce((a, b) => a + b, 0); |
There was a problem hiding this comment.
Would the function still work if line 19 is changed to .reduce((a, b) => a + b);?
cjyuan
added
Reviewed
Fithi-Teklom
added
Needs Review
2 participants
Self checklist