LappleApple / feedmereadmes

Free README editing+feedback to make your open source projects grow. See the README maturity model to help you keep going.
Creative Commons Attribution 4.0 International
1.13k stars 56 forks source link

Node Serialport #3

Closed reconbot closed 7 years ago

reconbot commented 7 years ago

Node Serialport is a widely used nodejs serialport library. I have users from all languages and countries, a lot of people new to javascript development and software development as a whole. To make it even more interesting we have at least 3 versions of serialport in wide use and the very old ones have a lot of bugs.

I have a few goals;

https://github.com/EmergingTechnologyAdvisors/node-serialport

LappleApple commented 7 years ago

Hey @reconbot, I'd edit this if I could skip over all the jsdoc stuff and do a simple fork-and-PR.

reconbot commented 7 years ago

What about it is preventing you?

LappleApple commented 7 years ago

I just want to edit in the browser

reconbot commented 7 years ago

If you start that way I'd be happy to help put the text in the right places. Jsdoc was invaluable in helping to have much better API documentation but it does spread out the API docs.

On Mon, May 1, 2017, 6:43 AM Lauri Apple notifications@github.com wrote:

I just want to edit in the browser

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/LappleApple/feedmereadmes/issues/3#issuecomment-298317539, or mute the thread https://github.com/notifications/unsubscribe-auth/AABlbo6Y1Y_48V3NpCZwWX0YJ_Sif5Qhks5r1bdfgaJpZM4Mw7iR .

LappleApple commented 7 years ago

That could work, @reconbot. I'll fork your repo, create an edited version of the README and then you can copy/paste the text into your README.md. Does that sound OK?

reconbot commented 7 years ago

Sounds good


Francis Gulotta wizard@roborooter.com

On Wed, May 3, 2017 at 7:55 AM, Lauri Apple notifications@github.com wrote:

That could work, @reconbot https://github.com/reconbot. I'll fork your repo, create an edited version of the README and then you can copy/paste the text into your README.md. Does that sound OK?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/LappleApple/feedmereadmes/issues/3#issuecomment-298890190, or mute the thread https://github.com/notifications/unsubscribe-auth/AABlbpQ79O2gpN7SXs_9Hvezun5bBr_sks5r2GsogaJpZM4Mw7iR .

LappleApple commented 7 years ago

Hey @reconbot, I've been busy flying around and traveling, but now that's over and I'll get to this by the weekend. Look for a PR from me. :)

LappleApple commented 7 years ago

@reconbot: I started editing. Everything above the TOC: https://github.com/LappleApple/node-serialport

I also moved the License section to the bottom and added the text. Will continue ...

reconbot commented 7 years ago

Feel free to make a WIP (work in progress) PR whenever you'd like some feedback. Looks good so far!

LappleApple commented 7 years ago

Thanks, @reconbot. I'll continue where I left off and finish. :)

LappleApple commented 7 years ago

@reconbot Is this line under the Alpine Linux section missing the end?

# If you're installing as root, you'll need to use

reconbot commented 7 years ago

That should refer to the --unsafe-perm flag mentioned in the sudo/root section https://github.com/EmergingTechnologyAdvisors/node-serialport#sudo--root

LappleApple commented 7 years ago

@reconbot Hi: I made a PR to let you know I'm ready for you to take over with the copy/pasting. :)