Skip to content

Contribution Requirement

Vitali Malinouski edited this page Dec 26, 2013 · 11 revisions

Your contributions are welcome and appreciated. However, I would like to lay down a few requirements to save your time (and mine) when you submit a pull request.

  1. Do not submit contributions with pure code-refactoring that provides no benefits of speed or features. I know you want the code look "your" way, so do I.

  2. Bug fixes, small enhancements, some refactoring (where it makes sense) are welcome and it is the way most people contribute.

  3. If you changes are involved or you want to add some awesome new feature, talk to me before coding. It might save you lot of energy talking to me afterwards in your pull request.

  4. Make sure your branch is up-to-date with current master.

  5. Follow same code-style guidelines used in the files

Style Guide

I like my code to look pretty, and for your contribution to make it you need to make it look pretty. You can use any general style guide (for example https://github.com/airbnb/javascript). In particular I also look for this

  1. Indentation with tabs (1 tab = 4 spaces)
  2. Align variables and properties nicely Good:
var obj = {
   prop1    : 1,
   otherVar : 2,
   prop3    : 3
}

Bad:

var obj = {
   prop1: 1,
   otherProp2: 2,
   prop3: 3
}
  1. Conditional statements Good:
if (some condition) {
	...
} else {
	...
}

Anything else is bad.

Clone this wiki locally