nodejs / nodejs-collection

A place for the Node.js Collective to organize the editing process
https://medium.com/the-node-js-collection/
MIT License
30 stars 16 forks source link

Node.js Collection: Submissions #15

Closed ZibbyKeaton closed 6 years ago

ZibbyKeaton commented 6 years ago

All -

As always thanks for your help here.

mhdawson commented 6 years ago

I read the last article by Safia and it seemed like a reasonable candidate to me.

WaleedAshraf commented 6 years ago

Safia articles are quite interesting. Might be too deep to understand for some people as she goes line by line of code, but she's doing a great job. ✋

Tiriel commented 6 years ago

I do like much Safia's articles. Indeed sometimes the explanations are quite short on some lines of code, but they are nonetheless very interesting.

ZibbyKeaton commented 6 years ago

Another article to think of adding to the collection: https://medium.com/@muehler.v/node-js-face-recognition-js-simple-and-robust-face-recognition-using-deep-learning-ea5ba8e852. Let me know if you agree with a yes 👍 or no 👎by Wednesday, January 17.

Trott commented 6 years ago

💯 on including Safia's series. (Haven't yet read the other stuff, so not saying anything about it here doesn't mean those articles aren't terrific as well.)

Trott commented 6 years ago

The bcrypt article seems A-OK to me on a quick pass, except that there are some spelling and grammar errors that are quite noticeable and make it a bit jarring to read. (guarentees instead of guarantees, specially instead of especially, more fast instead of faster, on experiment instead of an experiment...)

I'm happy to give it a technical review at some point soon, but can someone do a grammar/spelling clean-up first?

Trott commented 6 years ago

Content-wise, I am totally :+1: on the bcrypt article, although it would be good to get an N-API person to review it. Ping @nodejs/n-api

Trott commented 6 years ago

Facial-recognition article seems 👍 content-wise too. Like almost everything that gets submitted, it could definitely use some attention from an editor. (There's no shame in needing editorial review, of course. Even full-time professional writers need it. Writing is difficult.) If it helps, the issues in this one tend to be punctuation (missing commas, missing apostrophes, missing hyphenation) and capitalization (notably, euclidean should be Euclidean).

mhdawson commented 6 years ago

We were discussing this in the n-api meeting today and we are all +1 on the @NickNaso article (Note he is part of the team :))

ZibbyKeaton commented 6 years ago

Thanks for your feedback here @Trott I asked Nicola if I could go into this and edit for grammatical purposes.

mhdawson commented 6 years ago

@ZibbyKeaton do you know were we are on the N-API bcrypt article ?

ZibbyKeaton commented 6 years ago

It has been posted and I requested some grammatical suggestions @mhdawson

mhdawson commented 6 years ago

@ZibbyKeaton thanks for the update.

ZibbyKeaton commented 6 years ago

@WaleedAshraf I would like to post your blog on Wednesday. I just added you as a writer to the publication, so please submit it directly on Wednesday. Just go to the bottom of the article with the three dots and it will give you the option to submit it!

WaleedAshraf commented 6 years ago

Sure 👍 @ZibbyKeaton Thanks.