neon-bindings / examples

A collection of examples of Neon
324 stars 43 forks source link

Fix the broken explains in `Setup` section of `README.md` #50

Closed usagi closed 2 years ago

usagi commented 4 years ago

Problems:

  1. [ ] The different directories
  2. [ ] Is undefined the expected result?

Screenshot at the time of this reporting:

image

Details:

1. The different directories
git clone https://github.com/neon-bindings/examples
cd neon-examples

Maybe, the expected fixing is append neon-examples to the tail of the first line. Then it will be work:

git clone https://github.com/neon-bindings/examples neon-examples
cd neon-examples
2. Is undefined the expected result?
# Compile and run an example:
cd primitives
node ./lib/index.js

It will be returns undefined in Arch Linux(WSL 2) and Windows. At least, I gave up to use these examples for my studying NEON using this just as it is. I'll read the sources and debug first before run the example, it is very sadly situation for beginners. 😭

usagi commented 4 years ago

I could not understood these things at the first time of accessing this repos:


Of course, this might be a great training or implicit user filtering. But if so, I would be reconsider to use the project and contributing to this project. Be gentle for beginners and grow together. ❤️

amilajack commented 4 years ago

yes, undefined is the expected result. I agree that showing undefined can be confusing so I'll change the example. Thanks for the heads up!

usagi commented 4 years ago

Note: This issue to be simplify as the cd problem if the #53 merged. :tada: