1
0
Fork 0
mirror of https://github.com/lidarr/Lidarr synced 2024-12-23 00:03:05 +00:00
Lidarr/CONTRIBUTING.md
2015-02-20 21:14:18 -08:00

2.2 KiB

How to Contribute

We're always looking for people to help make NzbDrone even better, there are a number of ways to contribute. To get started, sign the Contributor License Agreement.

Documentation

Setup guides, FAQ, the more information we have on the wiki the better.

Development

Tools required

  • Visual Studio 2013
  • HTML/Javascript editor of choice (Sublime Text/Webstorm/etc)
  • npm (node package manager)
  • git

Getting started

  1. Fork NzbDrone
  2. Clone (develop branch)
  3. Run npm install
  4. Run gulp watch - Used to compile the UI components and copy them (leave this window open)
  5. Compile in Visual Studio

Contributing Code

  • If you're adding a new, already requested feature, please comment on Trello so work is not duplicated (If you want to add something not already on there, please talk to us first)
  • Rebase from NzbDrone's develop branch, don't merge
  • Make meaningful commits, or squash them
  • Feel free to make a pull request before work is complete, this will let us see where its at and make comments/suggest improvements
  • Reach out to us on the forums or on IRC if you have any questions
  • Add tests (unit/integration)
  • Commit with *nix line endings for consistency (We checkout Windows and commit *nix)
  • One feature/bug fix per pull request to keep things clean and easy to understand
  • Use 4 spaces instead of tabs, this is the default for VS 2012 and WebStorm (to my knowledge)

Pull Requesting

  • Only make pull requests to develop, never master, if you make a PR to master we'll comment on it and close it
  • You're probably going to get some comments or questions from us, they will be to ensure consistency and maintainability
  • We'll try to respond to pull requests as soon as possible, if its been a day or two, please reach out to us, we may have missed it
  • Each PR should come from its own feature branch not develop in your fork, it should have a meaningful branch name (what is being added/fixed)
    • new-feature (Good)
    • fix-bug (Good)
    • patch (Bad)
    • develop (Bad)

If you have any questions about any of this, please let us know.