"How to run our site on your machine?" section in README

FACK1 / languages-search

languages-search

0 stars 1 forks source link

Open mattlub opened 5 years ago

mattlub commented 5 years ago

This section is good! A couple of suggested improvements:

Use code blocks for the commands the user should run in the terminal: e.g.
```
git clone https://github.com/FACK1/languages-search
npm install
```
To run locally we don't need to run npm init (why not?)

orjwan98 commented 5 years ago

It's been taken care of. Please close this issue.

mattlub commented 5 years ago

@orjwan98 you have removed the npm install step (which is required)!

If this was raised in a PR instead of committed to master then maybe one of your teammates would have picked up on the error? 🤷‍♂️