Chart.js/docs/developers/contributing.md

Contributing

New contributions to the library are welcome, but we ask that you please follow these guidelines:

• Before opening a PR for major additions or changes, please discuss the expected API and/or implementation by filing an issue or asking about it in the Chart.js Slack #dev channel. This will save you development time by getting feedback upfront and make review faster by giving the maintainers more context and details.
• Consider whether your changes are useful for all users, or if creating a Chart.js plugin would be more appropriate.
• Check that your code will pass tests and eslint code standards. gulp test will run both the linter and tests for you.
• Add unit tests and document new functionality (in the test/ and docs/ directories respectively).
• Avoid breaking changes unless there is an upcoming major release, which are infrequent. We encourage people to write plugins for most new advanced features, and care a lot about backwards compatibility.
• We strongly prefer new methods to be added as private whenever possible. A method can be made private either by making a top-level function outside of a class or by prefixing it with _ and adding @private JSDoc if inside a class. Public APIs take considerable time to review and become locked once implemented as we have limited ability to change them without breaking backwards compatibility. Private APIs allow the flexibility to address unforeseen cases.

Joining the project

Active committers and contributors are invited to introduce yourself and request commit access to this project. We have a very active Slack community that you can join here. If you think you can help, we'd love to have you!

Building and Testing

Chart.js uses gulp to build the library into a single JavaScript file.

Firstly, we need to ensure development dependencies are installed. With node and npm installed, after cloning the Chart.js repo to a local directory, and navigating to that directory in the command line, we can run the following:

> npm install
> npm install -g gulp-cli

This will install the local development dependencies for Chart.js, along with a CLI for the JavaScript task runner gulp.

The following commands are now available from the repository root:

> gulp build                // build dist files in ./dist
> gulp build --watch        // build and watch for changes
> gulp unittest             // run tests from ./test/specs
> gulp unittest --watch     // run tests and watch for source changes
> gulp unittest --coverage  // run tests and generate coverage reports in ./coverage
> gulp lint                 // perform code linting (ESLint)
> gulp test                 // perform code linting and run unit tests
> gulp docs                 // build the documentation in ./dist/docs
> gulp docs --watch         // starts the gitbook live reloaded server

More information can be found in gulpfile.js.

Bugs and Issues

Please report these on the GitHub page - at github.com/chartjs/Chart.js. Please do not use issues for support requests. For help using Chart.js, please take a look at the chartjs tag on Stack Overflow.

Well structured, detailed bug reports are hugely valuable for the project.

Guidelines for reporting bugs:

• Check the issue search to see if it has already been reported
• Isolate the problem to a simple test case
• Please include a demonstration of the bug on a website such as JS Bin, JS Fiddle, or Codepen. (Template)

Please provide any additional details associated with the bug, if it's browser or screen density specific, or only happens with a certain configuration or data.