Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

north-west-5/kam-nik - #393

Open
KamNik0044 wants to merge 3 commits into
CodeYourFuture:masterfrom
KamNik0044:master
Open

north-west-5/kam-nik#393
KamNik0044 wants to merge 3 commits into
CodeYourFuture:masterfrom
KamNik0044:master

Conversation

@KamNik0044

Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

@SteveLeicester

Copy link
Copy Markdown

Thanks for submitting week one JS
Just a comment on the exercises - it is generally better to declare variables with let rather than var. Though you will continue to see var in older code modern practice is to use let. Certainly try to avoid mixing both together also.

The mandatory tests look good and pass so well done there.

Steve

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants