Skip to content

Cape Town | 26-ITP-Sep | Quentin Gwele | Sprint 2 | coursework/sprint-2-implement-and-rewrite - #1681

Open
lwandogwele52-ui wants to merge 6 commits into
CodeYourFuture:mainfrom
lwandogwele52-ui:coursework/sprint-3-implement-and-rewrite
Open

lwandogwele52-ui wants to merge 6 commits into
CodeYourFuture:mainfrom
lwandogwele52-ui:coursework/sprint-3-implement-and-rewrite

Conversation

@lwandogwele52-ui

Copy link
Copy Markdown

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

Task code

CYF-1059

Changelist

Did all necessary checks.

@lwandogwele52-ui lwandogwele52-ui added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 11, 2026
@lwandogwele52-ui lwandogwele52-ui changed the title Cape Town | 26-ITP-Sep | Quentin Gwele | Sprint 3 | coursework/sprint-3-implement-and-rewrite Cape Town | 26-ITP-Sep | Quentin Gwele | Sprint 2 | coursework/sprint-2-implement-and-rewrite Oct 12, 2026
@lwandogwele52-ui
lwandogwele52-ui deleted the coursework/sprint-3-implement-and-rewrite branch October 12, 2026 19:46
@lwandogwele52-ui
lwandogwele52-ui restored the coursework/sprint-3-implement-and-rewrite branch October 17, 2026 05:12
@hey-hammad hey-hammad added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Oct 17, 2026

@hey-hammad hey-hammad left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job. You’ve implemented the solution well and done a great job testing edge cases, including boundary values. The code is clean, readable, and easy to follow, and the test case names are clear and descriptive. Keep up the great work. Thanks

@lwandogwele52-ui

lwandogwele52-ui commented Oct 17, 2026 via email

Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants