Merge pull request #646 from HolgerJeromin/patch-2
contributing.md update with advice on commit messages and the current state of maintenance.
This commit is contained in:
commit
22094b1dd8
|
@ -1,5 +1,5 @@
|
|||
```
|
||||
Please note that AntennaPod is no longer maintained until further notice.
|
||||
Please note that AntennaPod is no longer maintained until further notice. See #604 for details about the future.
|
||||
```
|
||||
|
||||
How to report a bug
|
||||
|
@ -29,7 +29,7 @@ Submit a pull-request
|
|||
- If you want to work on a feature that has been requested or fix a bug that has been reported on the "issues" page, add a comment to it so that other people know that you are working on it.
|
||||
- Fork the repository
|
||||
- Almost all changes of AntennaPod are done on the `develop` branch. If a new version of AntennaPod is released, the `develop` branch is merged into `master`. As a result, the `master` branch probably doesn't contain the latest changes when you are reading this. Please make sure that you are branching from `develop`! Otherwise, there might be a lot of merge-conflicts when merging your changes into `develop` and therefore it might take longer to review your pull-request. Exceptions are changes in files like README.md, CONTRIBUTING.md and other files that are not directly related to the source code.
|
||||
- If your pull-request fixes a bug that has been reported or implements a feature that has been requested in another issue, try to mention it so that it can be closed once your pull-request has been merged.
|
||||
- If your pull-request fixes a bug that has been reported or implements a feature that has been requested in another issue, try to mention it so that it can be closed once your pull-request has been merged. If you use special keywords in the [commit comment](https://help.github.com/articles/closing-issues-via-commit-messages/) or [pull request text](https://github.com/blog/1506-closing-issues-via-pull-requests) github will close the issues automatically when the changes lands at the master branch.
|
||||
- Although not every part of AntennaPod's source code is fully documented yet, it would be very nice if you could add documentation to your changes if it is a larger pull-request.
|
||||
- If possible, add unit tests for your pull-request and make sure that they pass. Information on how to add unit tests and run them can be found here: [TODO]
|
||||
|
||||
|
|
Loading…
Reference in New Issue