Skip to content

London | 25-ITP-May | Hendrine Zeraua | Sprint 1 | Data Groups #635

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

rarityXtreme
Copy link

London | 25-ITP-May | Carol Owen | Sprint 1 | Alarm Clock

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

This PR covers my completed Sprint 1 tasks, divided into three key areas:
Fix: Debugged and corrected functions based on test feedback; all tests now pass.
Implement: Developed max, sum, and dedupe functions meeting the specified requirements and handling diverse inputs.
Refactor: Enhanced code readability and efficiency without breaking tests.
All tests were run using npm test, and the solutions adhere to the project’s structure.

Questions

I wanted to clarify whether the Stretch section of the coursework is optional. If so, I plan to prioritise the core tasks to stay on track and revisit the Stretch exercise at a later time when my schedule allows.

Thank you for your guidance.

@rarityXtreme rarityXtreme added the Needs Review Participant to add when requesting review label Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review Participant to add when requesting review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant