Skip to content

Brussels | ITP-2026-1 | Michael English | Sprint 1 | Coursework/sprint 1#64

Open
Eigo-G wants to merge 14 commits intoHackYourFutureBelgium:mainfrom
Eigo-G:M-S-and-T-D-sprints
Open

Brussels | ITP-2026-1 | Michael English | Sprint 1 | Coursework/sprint 1#64
Eigo-G wants to merge 14 commits intoHackYourFutureBelgium:mainfrom
Eigo-G:M-S-and-T-D-sprints

Conversation

@Eigo-G
Copy link
Copy Markdown

@Eigo-G Eigo-G commented Mar 21, 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

Briefly explain your PR.
Completed the exercises.

Questions

I changed the branch name on VSC but it hasn't updated on this end. Not sure why.
Ask any questions you have for your reviewer.

@@ -2,3 +2,5 @@

const age = 33;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There is an error when executing this code.
you can check if there are multiple declarations of the same variable

@@ -1,5 +1,8 @@
const cardNumber = 4533787178994213;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also there is an error when executing this code.
Maybe check if you are declaring things twice and if you are allowed to change constant variable types

@@ -1,2 +1,13 @@
const 12HourClockTime = "20:53";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The code is still failing,
The best approach is to execute it via Node. Let me know if you need help wiht that

@robertopaixao robertopaixao added the bug Something isn't working label Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants