Skip to content

London | 25-ITP-May | Kayanat Suleman | Sprint 3 | Dead Code#1022

Open
KayanatSuleman wants to merge 3 commits intoCodeYourFuture:mainfrom
KayanatSuleman:coursework/sprint-3-dead-code
Open

London | 25-ITP-May | Kayanat Suleman | Sprint 3 | Dead Code#1022
KayanatSuleman wants to merge 3 commits intoCodeYourFuture:mainfrom
KayanatSuleman:coursework/sprint-3-dead-code

Conversation

@KayanatSuleman
Copy link
Copy Markdown

@KayanatSuleman KayanatSuleman commented Feb 22, 2026

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

  • Removed unreachable/redundant code in Sprint-3/3-dead-code/exercise-1.js (removed unused variables + unreachable log).
  • Removed redundant intermediate variables/functions in Sprint-3/3-dead-code/exercise-2.js while keeping behaviour the same.

@KayanatSuleman KayanatSuleman added 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 22, 2026
@KayanatSuleman KayanatSuleman mentioned this pull request Feb 22, 2026
Open
10 tasks
@KayanatSuleman KayanatSuleman added the 📅 Sprint 3 Assigned during Sprint 3 of this module label Feb 22, 2026
This was referenced Feb 23, 2026
Closed
Open
hkavalikas
hkavalikas suggested changes Mar 1, 2026
View reviewed changes
@hkavalikas hkavalikas added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. 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. Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Mar 1, 2026
@KayanatSuleman KayanatSuleman added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Mar 3, 2026
@hkavalikas hkavalikas added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. 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. Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Mar 3, 2026
@hkavalikas
Copy link
Copy Markdown

hkavalikas commented Mar 3, 2026

Awesome work

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

Reviewers

1 more reviewer

@hkavalikas hkavalikas hkavalikas approved these changes

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KayanatSuleman @hkavalikas