Skip to content

London | 26-ITP-January| Mouawia Elkhalifa | Sprint 1 | Module Structuring and Testing Data#971

Closed
MouawiaElkhalifa wants to merge 2 commits into
CodeYourFuture:mainfrom
MouawiaElkhalifa:coursework/sprint-1
Closed

London | 26-ITP-January| Mouawia Elkhalifa | Sprint 1 | Module Structuring and Testing Data#971
MouawiaElkhalifa wants to merge 2 commits into
CodeYourFuture:mainfrom
MouawiaElkhalifa:coursework/sprint-1

Conversation

@MouawiaElkhalifa

Copy link
Copy Markdown

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

Completed all Sprint 1 exercises:

  • Key exercises
  • Mandatory errors
  • Mandatory interpret

This PR contains my completed Sprint 1 coursework, including all key exercises, mandatory errors, and mandatory interpret tasks. I tested each file using Node in the terminal and ensured the code follows the style guide.

Questions

No questions.

@MouawiaElkhalifa MouawiaElkhalifa added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 16, 2026
@A-O-Emmanuel A-O-Emmanuel added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Feb 21, 2026

@A-O-Emmanuel A-O-Emmanuel 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.

I left a comment.

@@ -1,5 +1,6 @@
// Currently trying to print the string "I was born in Bolton" but it isn't working...
// what's the error ?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the error?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The error was a ReferenceError because the variable cityOfBirth was used in the console.log before it was declared with const."

@A-O-Emmanuel A-O-Emmanuel Feb 21, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add it to your exercise answer, then commit it

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added the explanation

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still don't see it in your pull request

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay seen it

@A-O-Emmanuel A-O-Emmanuel added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Feb 21, 2026
@A-O-Emmanuel A-O-Emmanuel added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Feb 21, 2026
@illicitonion

Copy link
Copy Markdown
Member

Closing PR because the January ITP run has finished. Feel free to re-open if you're still working on it.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants