Skip to content

Commit

Permalink
Update Contributing instructions
Browse files Browse the repository at this point in the history
  • Loading branch information
ndbroadbent committed Dec 14, 2018
1 parent bff9c6e commit 3027b96
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
8 changes: 5 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -159,10 +159,12 @@ Before submitting a pull request, please check the following:
- Run `yarn test`
- 100% test coverage
- Coverage will be printed after running tests.
- Open the coverage results in your browser: `open coverage/lcov-report/index.html`
- Check the coverage results in your browser: `open coverage/lcov-report/index.html`
- No ESLint errors
- `yarn run lint`
- All code is formatted with [Prettier](https://prettier.io/)
- Run `prettier --write **/*.js`
- If you use VS Code, I recommend enabling the `formatOnSave` option.
- `yarn run format`
- If you use VS Code, you should enable the `formatOnSave` option.

## License

Expand Down
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
"scripts": {
"test": "jest",
"lint": "eslint --fix lib",
"format": "prettier --write **/*.js",
"update_deps": "node scripts/update_deps"
},
"repository": {
Expand Down

0 comments on commit 3027b96

Please sign in to comment.