Skip to content

London | ITP-26-May | Tomislav Dukez | Sprint 3 | Dead Code#1448

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

London | ITP-26-May | Tomislav Dukez | Sprint 3 | Dead Code#1448
tomdu3 wants to merge 3 commits into
CodeYourFuture:mainfrom
tomdu3:coursework/sprint-3/3-dead-code

Conversation

@tomdu3

@tomdu3 tomdu3 commented Jul 3, 2026

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

Changelist

My PR containes code related to:

  • exercise-1 and exercise-2 of the Dead Code tasks

@tomdu3 tomdu3 added 🏕 Priority Mandatory This work is expected 📅 Sprint 3 Assigned during Sprint 3 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 Jul 3, 2026

@LonMcGregor LonMcGregor 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.

Good work, task complete

@LonMcGregor LonMcGregor 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 Jul 8, 2026
@tomdu3

tomdu3 commented Jul 8, 2026

Copy link
Copy Markdown
Author

thanks for the review. you're awesome

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

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 3 Assigned during Sprint 3 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants