Skip to content

ZA | ITP-May | Dawud Vermeulen | Sprint-3 | Module-Structuring-and-Testing-Data #605

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: to-be-deleted-coursework/sprint-3
Choose a base branch
from
Open

ZA | ITP-May | Dawud Vermeulen | Sprint-3 | Module-Structuring-and-Testing-Data #605

wants to merge 9 commits into from
+7,621 −56

Conversation

Dave-Vermeulen
Copy link

Learners, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with REGION | COHORT_NAME | FIRST_NAME LAST_NAME | PROJ_NAME
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

  • Replaced incorrect line return str * count
  • Added validation to handle negative counts
  • Changed error message to match test expectations
  • Confirmed empty string comparison had no whitespace
  • Verified that the test file and function were saved and correctly referenced.

Questions

No questions.

@Dave-Vermeulen Dave-Vermeulen added the Needs Review Participant to add when requesting review label Jun 25, 2025
@Dave-Vermeulen Dave-Vermeulen linked an issue Jun 26, 2025 that may be closed by this pull request
02 Complete Sprint 3 coursework Dave-Vermeulen/My-Coursework-Planner#98
Open
@Waldo-Strydom
Copy link

well done

@Waldo-Strydom Waldo-Strydom added Reviewed Volunteer to add when completing a review Complete Volunteer to add when work is complete and review comments have been addressed and removed Needs Review Participant to add when requesting review labels Jun 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Complete Volunteer to add when work is complete and review comments have been addressed Reviewed Volunteer to add when completing a review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

02 Complete Sprint 3 coursework
3 participants
@Dave-Vermeulen @Waldo-Strydom @cjyuan