West-Midlands | 26-ITP-May | Maryam Janjua | Sprint 3 | Dead Code

[maryam-devio]

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 dead and redundant code while preserving the program's original behavior.

[jimoholayinka01]

Nice job removing the dead code instead of commenting it out that keeps the PR focused on the exercise. One thing to check though: you've removed the declaration of testName, but it's still being used later in the file.

testName = "Aman";

Since testName is no longer declared, this will cause an error. The goal is to remove only dead code while preserving the existing behaviour, so could you restore the declaration and only remove code that isn't used?

[maryam-devio]

Thank you for your feedback! You're absolutely right. My version wasn't running in strict mode, so I overlooked the missing declaration. I understand the importance of explicitly declaring variables with const and let for clarity and correctness.
I accidentally omitted const when declaring testName. I've corrected it now. Thank you for pointing it out!

[jimoholayinka01]

Well done. Your code is now error free and I will mark this as completed.