CONTRIBUTING.md

Bug Report

If you are submitting a bug report against a stable release, please check whether you have the latest release and your bug has been fixed in one of the newer versions. If it has been fixed, download the latest version and come back if the issue persists.

You should open a new issue ticket only if the issue has not been reported before. Please search the issue list, including all the closed bugs before you submit a new issue ticket. If the issue has been reported before, you should reopen the old one.

When reporting a new issue, you should use a succinct and concise subject line. In the body, you should write down the precise steps to reproduce the issues, and the browser vendor and version you are using.

Basic Email etiquette applies.

Feature Request

Please search the issue list first to see if your feature has already been requested. All features are tagged with the tag enhancement. If there already is an enchancement ticket, you can vote on it by putting in a +/-1 comment. If you cannot find the enhancement you are looking for, you can open a new issue.

Please prefix the subject line with [Feature Request] so your issue will stand out as a feature request. In the body, please lay out your rationale for such new feature to aid decision making.

Basic Email etiquette is appreciated.

Building

Backgrid.js uses Grunt and JSDuck <= 4.10.4 to generate its documentation.

Note:

You need to install node.js and npm first before you can install Grunt.

Once you have everything installed, you can do this to trigger a distribution build:

$ cd backgrid
$ npm install
# Recursively builds the core, extensions and put the output into lib
$ grunt dist
# Builds everything, including the docs and runs the tests
$ grunt

Branching

For every issue, there should be a branch for fixing that issue. When you are happy with your changes, you can file a pull request from your feature branch against Backgrid's master.

Code Style

If you are filing a pull request, you should follow the guidelines below:

Javascript

Please be aware that Backgrid.js uses 2 spaces for indentation. This project has a .jshintrc file defined, which is handy if you use jshint. In general, you should follow the Google Javascript Style Guide. Backgrid.js's code follows this coding style extremely closely, with 2 exceptions:

• else should be on its own line instead of the same line of the close } of the previous if block.
• " is preferred over ' in string literals because the project owner is cursed by C.

Testing

Backbone.js uses karma and jasmine for testing. You can invoke the following Grunt task to run the tests inside a terminal:

$ grunt karma

Commit Messages

This Erlang/OTP commit message guideline says it all. Take one minute to read it. If you are filing a pull request with commit messages that don't follow this guideline, don't fret, take another minute to rewrite them.

git log # find out how many commits you need to go back
git rebase -i HEAD~<number of changes to go back>
git push origin +<branch>:<branch>

Documentation

If you make any changes to the documentation, since Backgrid.js uses Github Pages, please do so in your gh-pages branch and submit a pull request from that branch.

If you change any Javascript comment documentation, you should rebuild the API document. There is also a Grunt task for this purpose, but make sure you have installed JSDuck first.

$ grunt doc

Changes that come with tests are more equal than others.

Pull requests that follow the above guidelines will get merged quicker than others.

Note:

Don't forget to keep your tree in sync with upstream before you submit a pull request. Usage of git-rebase is optional but encouraged.