London | 26-ITP-May | Gideon Defar | Sprint 1 | Coursework

[gideondefar]

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

In this PR, I have completed the required Sprint 1 coursework exercises.

• Resolved debugging and initialization order errors within the mandatory errors folder.
• Added comprehensive documentation and step-by-step breakdowns for the math logic (such as remainder operations and random integer formulas) and string manipulation files.
• Verified all JavaScript code runs cleanly using Node.js without terminal crashes.

Questions

How do you recommend formatting long multi-line explanations or breakdowns for formulas in the code? Is a detailed block comment preferred over individual single-line comments according to the style guide?

[webmonch]

I believe 0.js and 3.js in the mandatory errors section also require a solution. I also noticed that not all of the files run without errors. As a general practice, it’s worth running each file before pushing your changes to make sure there are no syntax or runtime errors.

[cjyuan]

Note: It seems this PR has been reviewed. So I changed to label to Reviewed.

[gideondefar]

I believe 0.js and 3.js in the mandatory errors section also require a solution. I also noticed that not all of the files run without errors. As a general practice, it’s worth running each file before pushing your changes to make sure there are no syntax or runtime errors.
Thanks for pointing that out. I’ve now fixed 0.js and 3.js and run all the files in the mandatory errors section to check for syntax and runtime errors.

[webmonch]

Good job! Most of the issues have been resolved, only few tiny ones left.

[gideondefar]

Good job! Most of the issues have been resolved, only few tiny ones left.

Could you please tell me a specific bug, so I can fix it. I couldn't see unresolved issue; I guess I have fixed all your feedback.

[cjyuan]

@gideondefar I think these two unresolved comments have not yet been addressed:
#1553 (comment)
#1553 (comment)

[gideondefar]

@gideondefar I think these two unresolved comments have not yet been addressed: #1553 (comment) #1553 (comment)

@cjyuan I have carefully go through them and resolved the issue thank you very much.

[cjyuan]

@gideondefar The commented-out code was not removed in your latest commit.

[gideondefar]

@gideondefar The commented-out code was not removed in your latest commit.

@cjyuan I don't know why it still display on his comment but it's gone on the file, I have double checked by fetching to my machine and push to the origin branch and from here when I click on edit file it doesn't exist. thanks!

[cjyuan]

I was referring to these two lines of commented out code.

[cjyuan]

It's an easy fix. I will mark this PR as "Complete" first.