Skip to content

Latest commit

 

History

History
34 lines (21 loc) · 2.27 KB

CONTRIBUTING.md

File metadata and controls

34 lines (21 loc) · 2.27 KB
Raw

Contributing guidelines

Pull request checklist

Before sending a pull request, be sure to follow this list.

How to become a contributor and submit your own code

We'd love to accept your changes, suggestions and patches! Be sure that your changes, source code, and other ideas/implementations do not cause intellectual property issues.

Contributing code

If you have any improvements or new functionality that is interesting for UrbanTrips, send us your pull requests! If you are new to pull requests, see Github's how to guide.

UrbanTrips team members will be assigned to review your pull requests. Once the pull requests are approved and pass continuous integration checks, a UrbanTrips team member will apply ready to pull label to your change. This means we are working on getting your pull request submitted to our internal repository. After the change has been submitted internally, your pull request will be merged automatically on GitHub.

Contribution guidelines and standards

Before sending your pull request for review, make sure your changes are consistent with the guidelines and follow the Google coding style.

General guidelines and philosophy for contribution

  • Include unit tests when you contribute new features, as they help to a) prove that your code works correctly, and b) guard against future breaking changes to lower the maintenance cost.
  • Bug fixes also generally require unit tests, because the presence of bugs usually indicates insufficient test coverage.
  • When you contribute a new feature to UrbanTrips, the maintenance burden is (by default) transferred to the UrbanTrips team. This means that the benefit of the contribution must be compared against the cost of maintaining the feature.
  • As every PR may require several CPU hours of CI testing, we discourage submitting PRs to fix one typo, one warning, etc. We recommend fixing the same issue at the file level at least (e.g.: fix all typos in a file, fix all compiler warning in a file, etc.)