Skip to content

Cape Town | 2026-ITP-Jan | Pretty Taruvinga | Sprint 1 | Mandatory errors#952

Open
Pretty548 wants to merge 5 commits intoCodeYourFuture:mainfrom
Pretty548:sprint1/mandatory-errors
Open

Cape Town | 2026-ITP-Jan | Pretty Taruvinga | Sprint 1 | Mandatory errors#952
Pretty548 wants to merge 5 commits intoCodeYourFuture:mainfrom
Pretty548:sprint1/mandatory-errors

Conversation

@Pretty548
Copy link

@Pretty548 Pretty548 commented Feb 10, 2026 •
edited
Loading

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

  1. Used const where the value needed to change → changed to let
  2. Declared a variable after using it → moved declaration above usage
  3. Tried to use .slice() on a number → converted to string first
  4. Duplicate variable declaration → removed the extra one
  5. Invalid variable name starting with a number → renamed it

Questions

Is there a clearer or more consistent way I should decide when to use const vs let in future exercises?

@github-actions

This comment has been minimized.

Sign in to view
4 similar comments
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 added 🏕 Priority Mandatory This work is expected 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Structuring-And-Testing-Data The name of the module. labels Feb 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
1 similar comment
@github-actions

This comment has been minimized.

Sign in to view
@github-actions github-actions bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
1 similar comment
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 changed the title Capetown|26-ITP-Jan|Pretty Taruvinga|Sprint1|mandatory errors Capetown|2026-ITP-Jan|Pretty Taruvinga|Sprint1|mandatory errors Feb 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@github-actions github-actions bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 10, 2026
@Pretty548 Pretty548 changed the title Capetown|2026-ITP-Jan|Pretty Taruvinga|Sprint1|mandatory errors Cape Town|2026-ITP-Jan|Pretty Taruvinga|Sprint1|mandatory errors Feb 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 changed the title Cape Town|2026-ITP-Jan|Pretty Taruvinga|Sprint1|mandatory errors Cape Town | 2026-ITP-Jan | Pretty Taruvinga | Sprint1 | Mandatory errors Feb 13, 2026
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 changed the title Cape Town | 2026-ITP-Jan | Pretty Taruvinga | Sprint1 | Mandatory errors Cape Town | 2026-ITP-Jan | Pretty Taruvinga | Sprint 1 | Mandatory errors Feb 13, 2026
@Pretty548 Pretty548 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 13, 2026
A-O-Emmanuel
A-O-Emmanuel reviewed Feb 14, 2026
View reviewed changes
Copy link

@A-O-Emmanuel A-O-Emmanuel left a comment

Choose a reason for hiding this comment

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

good job with this task, I don't have any other thing to add, other reviewers might have.

@A-O-Emmanuel A-O-Emmanuel added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Feb 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@A-O-Emmanuel A-O-Emmanuel A-O-Emmanuel left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed. Module-Structuring-And-Testing-Data The name of the module. 🏕 Priority Mandatory This work is expected 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Pretty548 @A-O-Emmanuel