-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #85 from TheBlackBit/contribution_guide
Create CONTRIBUTING.md
- Loading branch information
Showing
1 changed file
with
36 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
## How to contribute | ||
We'd love to accept your contributions to this App. There are just a few steps you need to follow. | ||
|
||
> [!IMPORTANT] | ||
> You must point your PR to the `develop` Branch. | ||
## Branching | ||
* fix/ -> Bugs, small fixes like typos. | ||
* feature/ -> New functionalities. | ||
* chore/ -> Changes that not affect the production code like update dependencies. | ||
* refactor/ -> Code refactor. | ||
* test/ -> Add missing test. | ||
|
||
|
||
## Pull request for review | ||
When you clone this repository a git hook is going to be installed in your local git. The git hooks are: | ||
* pre-commit. | ||
* pre-push. | ||
|
||
> [!NOTE] | ||
> If you are using a Windows maching review the scripts: | ||
> * scripts/pre-commit-windows. | ||
> * scripts/pre-push-windows. | ||
|
||
#### Pre-commit | ||
1. A script runs Ktlint format it is going to format your code before commit. | ||
2. A script runs all the unit tests to verify that none of the tests are broken. | ||
|
||
#### Pre-push | ||
1. A script runs all UI test locally. | ||
>[!WARNING] | ||
>YOU MUST HAVE A EMULATOR OR PHYSICAL DEVICE CONNECTED OR RUNNING. | ||
|
||
Please correct any failures before requesting a review. |