Manchester | 25-ITP-May | Fithi Teklom | Sprint 3 | 3-stretch

[Fithi-Teklom]

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

[github-actions]

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).

[github-actions]

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).

[cjyuan]

Branch is not clean (no wonder the code looks familiar)

[cjyuan]

Would the function still work if line 19 is changed to .reduce((a, b) => a + b);?

[Fithi-Teklom]

The function will not work correctly in all cases if you remove the initial value.
You should keep .reduce((a, b) => a + b, 0) to ensure consistent, safe behavior.

[cjyuan]

I was asking, will your function work differently if line 19 is changed from .reduce((a, b) => a + b, 0); to .reduce((a, b) => a + b);?

Can you find an example where your function would work differently?

Suggestion, look up how .reduce() work.

[cjyuan]

• The title of this PR is not quite correct.
• The PR description is missing a brief description of what has been changed in this PR branch.
• Branch is not clean.

Change is optional as this is a stretch exercise, but it’s never too early to develop the kind of attention to detail that defines great software developers.