Merge pull request #627 from fulldecent/patch-2

Separate CONTRIBUTING.md, see https://github.com/blog/1184-contributing-...
This commit is contained in:
Nick Downie 2014-09-17 15:04:30 +01:00
commit 1cfa42bc24
2 changed files with 11 additions and 11 deletions

11
CONTRIBUTING.md Normal file
View File

@ -0,0 +1,11 @@
Contributing
============
New contributions to the library are welcome, just a couple of guidelines:
* Tabs for indentation, not spaces please.
* Please ensure you're changing the individual files in /src, not the concatenated output in the Chart.js file in the root of the repo.
* Please check that your code will pass jshint code standards, gulp jshint will run this for you.
* Please keep pull requests concise, and document new functionality in the relevant .md file.
* Consider whether your changes are useful for all users, or if creating a Chart.js extension would be more appropriate.
* Please avoid committing in the build Chart.js & Chart.min.js file, as it causes conflicts when merging.

View File

@ -13,14 +13,3 @@ Chart.js is available under the [MIT license](http://opensource.org/licenses/MIT
## Bugs & issues
When reporting bugs or issues, if you could include a link to a simple [jsbin](http://jsbin.com) or similar demonstrating the issue, that'd be really helpful.
## Contributing
New contributions to the library are welcome, just a couple of guidelines:
- Tabs for indentation, not spaces please.
- Please ensure you're changing the individual files in `/src`, not the concatenated output in the `Chart.js` file in the root of the repo.
- Please check that your code will pass `jshint` code standards, `gulp jshint` will run this for you.
- Please keep pull requests concise, and document new functionality in the relevant `.md` file.
- Consider whether your changes are useful for all users, or if creating a Chart.js extension would be more appropriate.
- Please avoid committing in the build Chart.js & Chart.min.js file, as it causes conflicts when merging.