LFW211 Course Feedback

The course is great, but I think the code examples could be changed to improved visibility, maybe something like this?:

Regards.

Found a little typo in:

https://trainingportal.linuxfoundation.org/learn/course/nodejs-application-development-lfw211/nodes-event-system/nodes-event-system?page=2

Original code:

const { EventEmitter } = require('events')
class MyEmitter extends EventEmitter {
constructor (opts = {}) {
super(opts)
this.name = opts.name
},
destroy (err) {
if (err) { this.emit('error', err) }
this.emit('close')
}
}

Fixed (extra comma before "destroy"):

const { EventEmitter } = require('events')
class MyEmitter extends EventEmitter {
constructor (opts = {}) {
super(opts)
this.name = opts.name
--> }
destroy (err) {
if (err) { this.emit('error', err) }
this.emit('close')
}
}

Regards.

@a.taleghani said:
I want to THANK you! I was combing through my code wondering what I did wrong, if there was a typo on my end, frustrated so much until I looked through the course forum and saw that you are the ONLY person who posted about this! You saved me HOURS of headache and frustration! As a complete green (aside from the Intro Course Mr Clements also put together) I'm far away from being a senior engineer and likely wouldn't have caught this on my own and research would have been tedious. THANK YOU SO MUCH!!!!!!

Wow!, I'm glad that those posts helped you!, that was the intention, to help others go through the course with less frustration facing these issues. You motivated me to keep doing it, thank you too for your message. Regards!.

On module 15, step "Process Configuration (1)" (https://trainingportal.linuxfoundation.org/learn/course/nodejs-application-development-lfw211/creating-child-processes/creating-child-processes?page=9) the example that should replace the env variables in the child process, well, doesn't, it shows the new defined variable "SUBPROCESS_SPECIFIC" among other variables:

Maybe this is because of my Windows environment?, I should try in linux.

On module 16, on "jest Framework: test/req.test.js" step (https://trainingportal.linuxfoundation.org/learn/course/nodejs-application-development-lfw211/writing-unit-tests/writing-unit-tests?page=12) there is no npm command that installs jest framework, so I'm guessing that this command is missing:

npm install --save-dev jest

test file runs after installing jest.