Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

Latest commit

 

History

History
61 lines (42 loc) · 1.66 KB

CONTRIBUTING.md

File metadata and controls

61 lines (42 loc) · 1.66 KB

Contributing to Thorn

Think twice, code once.

Pull requests are accepted under the discretion of Thorn maintainer(s). Improve the chances of your pull requests getting merged by following the requirements and guidelines below. For any questions, please e-mail developers@sugarcrm.com.

Requirements

Guidelines

  • Make sure your pull request contains quality code. We will certainly provide constructive feedback on works in progress but we will not merge incomplete pull requests.
  • Make sure your pull requests are properly tested and maintain (or improve) the current code coverage.
  • Make sure your pull request is fully documented.
  • Reference related Github issues within commit messages and pull request comments where appropriate.

Bugs

Setup

You'll need Node.js, yarn, and gulp-cli installed.

$ git clone https://github.com/sugarcrm/thorn
$ yarn

You should also add dist to your .git/info/exclude file. It is not included in .gitignore because this interferes with properly publishing Thorn as an npm module.

$ echo "dist" >> .git/info/exclude

Compiling

$ gulp

Linting

$ gulp lint

Building the Documentation

$ gulp doc

Testing

$ gulp test [--coverage]