search

neutralinojs / neutralinojs-cli

neu cli for Neutralinojs
https://neutralino.js.org/docs/cli/neu-cli
MIT License
91 stars 57 forks source link

Fix #161 #162

Closed pegvin closed 2 years ago

pegvin commented 2 years ago

Fixes Issue #161 By Using Optional Chaining Operator to read the value of a property located deep within a chain of connected objects without having to check that each reference in the chain is valid.

pathange-s commented 2 years ago

Ahh, we can have a better description for the pull request ;) (but seems like it will get a quick merge, so yeah)

pathange-s commented 2 years ago

@shalithasuranga this LGTM. Also, the tests further written will depend on this. Please merge this pull request.

Wiiseguy commented 2 years ago

I manually applied this fix in my global node_modules and it fixes a crash I encountered when running neu run --frontend-lib-dev. It would be really nice if this fix could be published to NPM as a 9.3.x patch soon.