Skip to content

Instantly share code, notes, and snippets.

@chimericdream
Forked from johnmdonahue/contributing.md
Last active August 29, 2015 14:24
Show Gist options
  • Save chimericdream/feee53362fcb3ec22d59 to your computer and use it in GitHub Desktop.
Save chimericdream/feee53362fcb3ec22d59 to your computer and use it in GitHub Desktop.

Contributing to EpicEditor

Suggestions for contributing to EpicEditor. For feature and bug fix contributions we try to follow Vincent Driessen's git branching model. No fix or feature is too small. Thanks for contributing.

Issues

Before submitting a bug report or feature request, please check to make sure it hasn't already been submitted. You can indicate support for an existing issue by commenting on that issue. When submitting a bug, please include any details necessary to reproduce it (e.g. browser, OS etc). And if you're feeling frisky, including a failing test is super helpful.

Pull requests

If you are fixing a bug you found or adding a feature, please report the bug or feature and take note of the ticket number. After fixing the bug use the following template for your commit messages:

$ git commit -a -m "Ticket #[ticket number] - Fixes foo"

This makes it easier to track stuff down later on.

Requirements

We rely on Jake to manage tasks, foounit for tests and JSHint for linting. All of these run on NodeJS and can be installed via npm as follows:

$ sudo npm install -g jake foounit jshint

Flow

  1. Fork the project.
  2. Create a topic branch e.g. git checkout -b epic-bug-fix git checkout -b new-feature.
  3. Add tests where possible.
  4. Implement your feature or bug fix. Changes should be made to the src/ files, not the built files in epiceditor/.
  5. Run tests: jake lint and jake test.
  6. Rebuild: If everything is passing, jake build.
  7. Commit and push your changes. Try to reference the associated ticket number in your commit message as noted above.
  8. Submit your pull request - ideally targeting the development branch

A core developer will add a black label with a version number where this was pulled in, i.e. if the fix was included 0.1.1 then we will add a black label with 0.1.1 and close the ticket. Closed tickets do not mean they are in the release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment