|
| 1 | +Contributing Guidelines |
| 2 | +-------------------------------------------------- |
| 3 | + |
| 4 | +This document provides general guidelines about how to contribute to the project. Keep in mind these important things before you start contributing. |
| 5 | + |
| 6 | +### Asking Questions |
| 7 | + |
| 8 | +We do not use github issues for general library support. We think this questions should be posted on stack overflow using [GenericPasswordRow](http://http://stackoverflow.com/questions/tagged/GenericPasswordRow) tag. |
| 9 | + |
| 10 | +### Reporting issues |
| 11 | + |
| 12 | +* Use [github issues](https://github.com/EurekaCommunity/GenericPasswordRow/issues) to report a bug. |
| 13 | +* Before creating a new issue: |
| 14 | + * Make sure you are using the [latest release](https://github.com/EurekaCommunity/GenericPasswordRow/releases). |
| 15 | + * Check if the issue was [already reported or fixed](https://github.com/EurekaCommunity/GenericPasswordRow/issues?utf8=%E2%9C%93&q=is%3Aissue). Notice that it may not be released yet. |
| 16 | + * If you found a match add a brief comment "I have the same problem" or "+1". This helps prioritize the issues addressing the most common and critical first. If possible add additional information to help us reproduce and fix the issue. Please use your best judgement. |
| 17 | +* Reporting issues: |
| 18 | + * Please include the following information to help maintainers to fix the problem faster: |
| 19 | + * Xcode version you are using. |
| 20 | + * iOS version you are targeting. |
| 21 | + * Full Xcode console output of stack trace or code compilation error. |
| 22 | + * Any other additional detail you think it would be useful to understand and solve the problem. |
| 23 | + |
| 24 | + |
| 25 | +### Pull requests |
| 26 | + |
| 27 | +The easiest way to start contributing is searching open issues by `help wanted` tag. We also add a `difficulty` tag (difficulty: easy, difficulty: moderate, difficulty: hard) in order to give an idea of how complex it can be to implement the feature according maintainers project experience. |
| 28 | + |
| 29 | +* Add test coverage to the feature or fix. We only accept new feature pull requests that have related test coverage. This allows us to keep the library stable as we move forward. |
| 30 | +* Remember to document the new feature. We do not accept new feature pull requests without its associated documentation. |
| 31 | +* In case of a new feature please update the example project showing the feature. |
| 32 | +* Please only one fix or feature per pull request. This will increase the chances your feature will be merged. |
| 33 | + |
| 34 | + |
| 35 | +###### Suggested git workflow to contribute |
| 36 | + |
| 37 | +1. Fork the GenericPasswordRow repository. |
| 38 | +2. Clone your forked project into your developer machine: `git clone [email protected]:<your-github-username>/GenericPasswordRow.git` |
| 39 | +3. Add the original project repo as upstream repository in your forked project: `git remote add upstream [email protected]:EurekaCommunity/GenericPasswordRow.git` |
| 40 | +4. Before starting a new feature make sure your forked master branch is synchronized upstream master branch. Considering you do not mere your pull request into master you can run: `git checkout master` and then `git pull upstream master`. Optionally `git push origin master`. |
| 41 | +5. Create a new branch. Note that the starting point is the upstream master branch HEAD. `git checkout -b my-feature-name` |
| 42 | +6. Stage all your changes `git add .` and commit them `git commit -m "Your commit message"` |
| 43 | +7. Make sure your branch is up to date with upstream master, `git pull --rebase upstream master`, resolve conflicts if necessary. This will move your commit to the top of git stack. |
| 44 | +8. Squash your commits into one commit. `git rebase -i HEAD~6` considering you did 6 commits. |
| 45 | +9. Push your branch into your forked remote repository. |
| 46 | +10. Create a new pull request adding any useful comment. |
| 47 | + |
| 48 | + |
| 49 | +###### Code style and conventions |
| 50 | + |
| 51 | +We try to follow our [swift style guide](https://github.com/EurekaCommunity/Swift-Style-Guide). Following it is not strictly necessary to contribute and to have a pull request accepted but project maintainers try to follow it. We would love to hear your ideas to improve our code style and conventions. Feel free to contribute. |
| 52 | + |
| 53 | + |
| 54 | +### Feature proposal |
| 55 | + |
| 56 | +We would love to hear your ideas and make a discussions about it. |
| 57 | + |
| 58 | +* Use github issues to make feature proposals. |
| 59 | +* We use `type: feature request` label to mark all [feature request issues](https://github.com/EurekaCommunity/GenericPasswordRow/labels/type%3A%20feature%20request). |
| 60 | +* Before submitting your proposal make sure there is no similar feature request. If you found a match feel free to join the discussion or just add a brief "+1" if you think the feature is worth implementing. |
| 61 | +* Be as specific as possible providing a precise explanation of feature request so anyone can understand the problem and the benefits of solving it. |
0 commit comments