Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update readme #123

Open
mayeaux opened this issue Apr 24, 2016 · 4 comments
Open

Update readme #123

mayeaux opened this issue Apr 24, 2016 · 4 comments

Comments

@mayeaux
Copy link
Member

mayeaux commented Apr 24, 2016

Explain bower is a dep for npm install
Explain using electron-prebuilt

Add new developers
Change the link directly to the heroku app for getting a slack invite

@SaturnusDJ
Copy link
Member

Yeah if bower is not automatically installed by npm install it should be explained. Then again, @hanford was working on removing bower so might not be needed to explain bower.

@mayeaux
Copy link
Member Author

mayeaux commented Apr 24, 2016

Takes only 30 seconds to add it to the readme. I'll get this done sometime soon

@amosshapira
Copy link

amosshapira commented Jan 3, 2017

Yep, I'm not familiar with the npm eco-system and had to figure out to manually execute npm install bower from the error message I got from npm install while I was following the instructions in the README.

I don't see any downside for adding this step until Bower dependency is removed.

@gdborton
Copy link
Contributor

gdborton commented Mar 2, 2017

I believe that this can be closed. I see that bower is in the README.

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

No branches or pull requests

4 participants