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

WM3 - Peter Ahern - JS2Week1#142

Open
PeterAhern wants to merge 2 commits intoCodeYourFuture:mainfrom
PeterAhern:main
Open

WM3 - Peter Ahern - JS2Week1#142
PeterAhern wants to merge 2 commits intoCodeYourFuture:mainfrom
PeterAhern:main

Conversation

@PeterAhern
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: Peter Ahern
  • Your City: Birmingham
  • Your Slack Name: PeterAhern

Homework Details

  • Module: JSCore2
  • Week:1

Notes

  • What did you find easy?

  • What did you find hard?

  • Found a lot of it very challenging and time consuming, but it is because I have got behind and returning to this was difficult to remember the material.

  • What do you still not understand?

  • Any other notes?

PeterAhern and others added 2 commits October 18, 2021 16:36
A few errors left when running some tests and 2 exercises to finsih
Comment thread mandatory/1-writers.js
array.forEach(function (array) {
if (array.age >= 40 && array.age <= 49 && array.alive === true) {
console.log(
`Hi, my name is ${array.firstName} ${array.lastName}. I am ${array.age} years old.`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

well done here...clean code

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