1
0
mirror of https://github.com/ultrasonic/ultrasonic synced 2024-12-12 16:57:23 +01:00
ultrasonic-app-subsonic-and.../CONTRIBUTING.md
Yahor Berdnikau 385790d601 Update doc with what branch use by default for PR.
Signed-off-by: Yahor Berdnikau <egorr.berd@gmail.com>
2017-12-16 22:28:31 +01:00

1.4 KiB
Raw Blame History

Contributing

Ultrasonic development is a community project, and contributions are welcomed.

First, see if your issue havent been yet reported here, then, please, first discuss the change you wish to make via a new issue.

By default Pull Request should be opened against develop branch, PR against master branch should be used only for critical bugfixes.

Here are a few guidelines you should follow before submitting:

  1. License Acceptance: All contributions must be licensed as GNU GPLv3 to be accepted. Use git commit --signoff to acknowledge this.
  2. App is migrating to Kotlin programming language: new Pull Requests should be written in this programming language.
  3. No Breakage: New features or changes to existing ones must not degrade the user experience.
  4. Coding standards best-practices should be followed, comment generously, and avoid "clever" algorithms. Refactoring existing messes is great, but watch out for breakage.
  5. No large PR:* Try to limit the scope of PR only to the related issue, so it will be easier to review and test.

Pull Request Process

  1. Ensure all commits are signed-off.
  2. Check tests for the new code are added.
  3. Check code style is passing.
  4. Check code static analysis is passing.