Skip to content

Latest commit

 

History

History
74 lines (49 loc) · 5.65 KB

CONTRIBUTING.md

File metadata and controls

74 lines (49 loc) · 5.65 KB
Raw

Contribute to OneBusAway Alexa

This guide details how to use issues and pull requests (for new code) to improve OneBusAway Alexa.

Individual Contributor License Agreement (ICLA)

To ensure that the app source code remains fully open-source under a common license, we require that contributors sign an electronic ICLA before contributions can be merged. When you submit a pull request, you'll be prompted by the CLA Assistant to sign the ICLA.

Code of Conduct

This project adheres to a Code of Conduct. By participating, you are expected to honor this code.

Code Style and Template

See the main OneBusAway Code Style Guide.

Closing policy for issues and pull requests

OneBusAway is a popular project and the capacity to deal with issues and pull requests is limited. Out of respect for our volunteers, issues and pull requests not in line with the guidelines listed in this document may be closed without notice.

Please treat our volunteers with courtesy and respect, it will go a long way towards getting your issue resolved.

Issues and pull requests should be in English and contain appropriate language for audiences of all ages.

Issue tracker

The issue tracker is only for obvious bugs, misbehavior, & feature requests in the latest stable or development release of OneBusAway Alexa. When submitting an issue please conform to the issue submission guidelines listed below. Not all issues will be addressed and your issue is more likely to be addressed if you submit a pull request which partially or fully addresses the issue.

Issue tracker guidelines

Search for similar entries before submitting your own, there's a good chance somebody else had the same issue or feature request. Show your support with :+1: and/or join the discussion. Please submit issues in the following format (as the first post) and feature requests in a similar format:

  1. Summary: Summarize your issue in one sentence (what goes wrong, what did you expect to happen)
  2. Steps to reproduce: How can we reproduce the issue?
  3. Expected behavior: What did you expect the app to do?
  4. Observed behavior: What did you see instead? Describe your issue in detail here.
  5. Possible fixes: If you can, link to the line of code that might be responsible for the problem.

Pull requests

We welcome pull requests with fixes and improvements to OneBusAway Alexa code, tests, and/or documentation. The features we would really like a pull request for are open issues with the enhancements label.

Pull request guidelines

If you can, please submit a pull request with the fix or improvements including tests. If you don't know how to fix the issue but can write a test that exposes the issue we will accept that as well. In general bug fixes that include a regression test are merged quickly while new features without proper tests are least likely to receive timely feedback. The workflow to make a pull request is as follows:

  1. Sign the ICLA (first time only)
  2. Fork the project on GitHub
  3. Create a feature branch
  4. Write tests and code
  5. Run the unit tests with mvn test to make sure you didn't break anything
  6. Apply the OneBusAway Code Style Guide to your code in Eclipse.
  7. If you created new files, add the Apache license header to them (see this file for an example). If you modified an existing file, add your name to the copyright attribution in the header.
  8. If you have multiple commits please combine them into one commit by squashing them. See this article and this Git documentation for instructions.
  9. Push the commit to your fork
  10. Submit a pull request with a motive for your change and the method you used to achieve it
  11. Search for issues related to your pull request and mention them in the pull request description or comments

We will accept pull requests if:

  • The code has proper tests and all tests pass (or it is a test exposing a failure in existing code)
  • It can be merged without problems (if not please use: git rebase master)
  • It doesn't break any existing functionality
  • It's quality code that conforms to standard style guides and best practices
  • The description includes a motive for your change and the method you used to achieve it
  • It is not a catch all pull request but rather fixes a specific issue or implements a specific feature
  • It keeps the OneBusAway Alexa code base clean and well structured
  • We think other users will benefit from the same functionality
  • If it makes changes to the UI the pull request should include screenshots
  • It is a single commit (please use git rebase -i to squash commits)

License

By contributing code to this project via pull requests, patches, or any other process, you are agreeing to license your contributions under the Apache License, Version 2.0.