puppeteer/CONTRIBUTING.md

56 lines
1.9 KiB
Markdown
Raw Normal View History

2017-05-11 07:06:41 +00:00
# How to Contribute
2017-06-19 23:56:29 +00:00
First of all, thank you for your interest in Puppeteer!
We'd love to accept your patches and contributions!
2017-05-11 07:06:41 +00:00
## Contributor License Agreement
Contributions to this project must be accompanied by a Contributor License
Agreement. You (or your employer) retain the copyright to your contribution,
this simply gives us permission to use and redistribute your contributions as
part of the project. Head over to <https://cla.developers.google.com/> to see
your current agreements on file or to sign a new one.
You generally only need to submit a CLA once, so if you've already submitted one
(even if it was for a different project), you probably don't need to do it
again.
## Code reviews
All submissions, including submissions by project members, require review. We
use GitHub pull requests for this purpose. Consult
[GitHub Help](https://help.github.com/articles/about-pull-requests/) for more
information on using pull requests.
2017-06-19 23:56:29 +00:00
## Code Style
The coding style is fully defined in [.eslintrc](https://github.com/GoogleChrome/puppeteer/blob/master/.eslintrc.js).
Please make sure to run `npm lint` before submitting PR.
We use JSDoc along with closure annotations. Annotations are encouraged for
all contributions.
## Testing
All new features should be accompanied by tests. Puppeteer tests are located in [test/test.js](https://github.com/GoogleChrome/puppeteer/blob/master/test/test.js)
and are written using [Jasmine](https://jasmine.github.io/) testing framework.
There are also phantomjs tests located under [third_party/phantomjs/test](https://github.com/GoogleChrome/puppeteer/tree/master/third_party/phantomjs). These
are used to test `phantom_shim`.
To run puppeteer tests, use:
```
npm run test-puppeteer
```
To run phantom-shim against phantomjs tests, use:
```
npm run test-phantom
```
To run both puppeteer and phantom_shim tests, use:
```
npm test
```