London | 26-ITP-January | Laura C | Sprint 1 | Data Groups

[lauracs24]

Learners, PR Template

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

Changelist

• Completed Fix section (calculateMedian)
• Implemented max, sum, and dedupe using TDD
• Refactored includes to use a for...of loop
• Ensured all tests pass (fix, implement, refactor)
• Made small, incremental commits throughout the work

[JaypeeLan]

Number.isFinite() already ensures it is a number, so you don't need to check the type.
Also, the filter is not needed, you can do the check in the for of loop, so you don't have 2 loops.

[lauracs24]

Thanks for the suggestion! I've refactored the implementation to remove the filter and perform the Number.isFinite() check directly inside the for of loop.