Skip to content

Commit

Permalink
fix: Updating CONTRIBUTING guide to include correct script references
Browse files Browse the repository at this point in the history
  • Loading branch information
rhyswfbowles committed Oct 9, 2020
1 parent 5c00fd8 commit 77ed834
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
- Make sure `npm test` passes. (see [development setup](#development-setup))

- If adding new feature:

- Add accompanying test case.
- Provide convincing reason to add this feature. Ideally you should open a suggestion issue first and have it greenlighted before working on it.

Expand All @@ -41,6 +42,7 @@
- Add appropriate test coverage if applicable.

### Work Step Example

- Fork the repository from [nicolasbeauvais/vue-social-sharing](https://github.com/nicolasbeauvais/vue-social-sharing) !
- Create your topic branch from `master`: `git branch my-new-topic origin/master`
- Add codes and pass tests !
Expand All @@ -59,21 +61,23 @@ After cloning the repo, run:
### Commonly used NPM scripts

# watch and serve with hot reload unit test at localhost:8080
$ npm run dev
$ npm run storybook

# lint source codes
$ npm run lint

# run unit tests in browser (firefox/safari/chrome)
$ npm run test:unit
$ npm run test

# build all dist files, including npm packages
$ npm run build

# run the full test suite, include linting
$ npm test

# render an example page with all the buttons
$ npm run example

There are some other scripts available in the `scripts` section of the `package.json` file.

The default test script will do the following: lint with ESLint -> unit tests with coverage -> e2e tests. **Please make sure to have this pass successfully before submitting a PR.** Although the same tests will be run against your PR on the CI server, it is better to have it working locally beforehand.

0 comments on commit 77ed834

Please sign in to comment.