Skip to content

Commit 49ecd1c

Browse files
committed
install gulp as dev dependency
1 parent dadaf8a commit 49ecd1c

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

README.md

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,19 +29,23 @@ When run through the workshop the low level code to talk to the Arduino is stubb
2929

3030
Wire up an Arduino, connect the USB and run your solution directly to see it run for real.
3131

32-
## Code style
32+
## Contributing
33+
34+
### Code style
3335

3436
To move towards a consistent style for nodeschool projects we use the .jshintrc
3537
as defined in learnyounode: https://github.com/rvagg/learnyounode/blob/master/.jshintrc
3638

37-
Your favourite jshint runner will work but a gulpfile is provided for hipsters.
39+
### Building
3840

39-
```shell
40-
npm install -g gulp
41-
gulp
41+
npm scripts are present for testing and linting:
42+
43+
```
44+
$ npm run test
45+
$ npm run lint
4246
```
4347

44-
...to run it.
48+
`pre-commit` hooks are installed in the repo to run the test/lint scripts with every commit.
4549

4650
## Thanks
4751

package.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,14 +48,15 @@
4848
"node-notifier": "^4.3.1"
4949
},
5050
"devDependencies": {
51-
"gulp": "^3.6.2",
51+
"gulp": "^3.9.1",
5252
"gulp-jshint": "^2.0.0",
5353
"gulp-shell": "^0.5.1",
5454
"jshint": "^2.8.0",
5555
"jshint-stylish": "^2.1.0",
5656
"pre-commit": "^1.1.2"
5757
},
5858
"pre-commit": [
59-
"lint", "test"
59+
"lint",
60+
"test"
6061
]
6162
}

0 commit comments

Comments
 (0)