This repository has been archived by the owner on Jan 19, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 17
Contributing
Alex Vernacchia edited this page Jun 27, 2015
·
5 revisions
We welcome all contributions! We just ask that you follow some simple guidelines.
- Use tabs, not spaces
- Before all pull requests:
- Run
grunt jshint
before submitting pull requests- Checkout our
.jshintrc
for the conventions we use
- Checkout our
- Rebase commits into manageable chunks relevant to contributed functionality
- Please use descriptive commit messages. Take a look at the projects past commit messages to get a feel
-
Write tests, if you've added functionality that the current tests don't cover
- Tests are written in mocha. To generate coverage -->
istanbul cover _mocha test/
- I've include the test coverage below. It should stay the same, or go up (never down)
- Tests are written in mocha. To generate coverage -->
- Run
Coverage as of June 27th, 2015
=============================== Coverage summary ===============================
Statements : 100% ( 68/68 )
Branches : 98% ( 49/50 )
Functions : 100% ( 11/11 )
Lines : 100% ( 68/68 )
================================================================================