Skip to content

Latest commit

 

History

History
74 lines (45 loc) · 3.29 KB

CONTRIBUTING.md

File metadata and controls

74 lines (45 loc) · 3.29 KB

Contributing to Ractive.js

Many thanks for using Ractive and contributing to its development. The following is a quick set of guidelines designed to maximise your contribution's effectiveness.

Reporting security vulnerabilities

If you think you've found a security vulnerability, please email [email protected] with details, and someone from the core team will respond to you.

Got a question or need help?

If you're having trouble getting Ractive to do what you want, there are a couple of places to get help:

Raising issues

Before submitting an issue, please make sure you're using the latest released version. A copy of the latest can be found at http://cdn.ractivejs.org/latest/ractive.js.

If the bug persists, it may have been fixed in the latest development version. You can always get the most recent successful build from http://cdn.ractivejs.org/edge/ractive.js.

The best issues contain a reproducible demonstration of the bug in the form of a JSFiddle or similar. This fiddle has a basic setup to get started with. Even better, you could create a failing test case using this fiddle as a base.

Pull requests

Pull requests against the master branch will not be merged!

All pull requests are welcome. You should create a new branch, use the dev branch as base, and submit the PR against the dev branch.

Before submitting, run npm run build which will concatenate, lint and test the code to ensure the build passes. Don't include files from outside the src and test folders in the PR.

If in doubt, submit the PR. A PR that needs tweaking is infinitely more valuable than a request that wasn't made because you were worrying about meeting these requirements.

Code Style Guide

There isn't (yet) a formal style guide for Ractive, so please take care to adhere to existing conventions:

  • Tabs, not spaces!
  • Variables at the top of function declarations
  • Semi-colons
  • Single-quotes for strings
  • Liberal whitespace:
// this...
var foo = function ( bar ) {
	var key;

	for ( key in bar ) {
		doSomething( bar, key ); // no space between function name and bracket for invocations
	}
};

// ...NOT this
var foo = function(bar){
	for(var key in bar){
		doSomething(bar, key);
	}
}

Above all, code should be clean and readable, and commented where necessary. If you add a new feature, make sure you add a test to go along with it!

Contributor License Agreement

There's no contributor license agreement. Contributions are made on a common sense basis. Ractive is distributed under the MIT license, which means your contributions will be too.