Skip to content

Cape Town | 26-ITP- Sept | Leigh Ross | Sprint 3 | Refactoring Dead Code - #1719

Open
leigh-ross wants to merge 9 commits into
CodeYourFuture:mainfrom
leigh-ross:coursework/sprint-3-dead-code
Open

leigh-ross wants to merge 9 commits into
CodeYourFuture:mainfrom
leigh-ross:coursework/sprint-3-dead-code

Conversation

@leigh-ross

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-1066

Changelist

Removed all dead code from exercise-1 and exercise-2.

@leigh-ross leigh-ross added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 16, 2026
Comment thread dead-code/exercise-1.js Resolved
Comment thread dead-code/exercise-1.js Resolved
@KudahShambare KudahShambare added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Oct 21, 2026
@leigh-ross leigh-ross added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 27, 2026
@KudahShambare KudahShambare 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. Complete Volunteer to add when work is complete and all review comments have been addressed. labels Nov 3, 2026
@leigh-ross

Copy link
Copy Markdown
Author

@KudahShambare if my work is satisfactory could you kindly add the complete label? Much thanks

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

Labels

Reviewed Volunteer to add when completing a review with trainee action still to take.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants