Skip to content
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.

updated README with Quick start guide #134

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

hygull
Copy link

@hygull hygull commented Jan 9, 2019

I found your package very useful in comparison to nw-builder and thought to add a Quick start guide.

As I found, most of the node users with little experience don't know about scripts available in package.json file.

So running npm run dist is like finding/looking for 1 more help to run the project and package the NW app.

I have added 2 more simple markdown files for users who will be looking for real examples generated by npm init command.

@TheJaredWilcurt
Copy link

There's a lot of superfluous content in here that could slow down or confuse new users. I'll add notes later today.

@TheJaredWilcurt
Copy link

So I'm looking at it now, to make comments, but it looks as though you've basically removed all of the changes. I don't know if that was on purpose or if you were offended by the term "superfluous". I was referring to things like having 25 keywords in the example package.json as not being necessary.

@hygull
Copy link
Author

hygull commented Jan 12, 2019

Yeah, I thought the lines I had added were really not useful, so I removed them again. Anyway thank you for your response.

Copy link

@TheJaredWilcurt TheJaredWilcurt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants