Merge pull request #55 from status-im/contributing/commit-message
Added comments about commit message
This commit is contained in:
commit
0b4c48564f
|
@ -118,9 +118,14 @@ The following categories are;
|
||||||
|
|
||||||
### Pull Requests
|
### Pull Requests
|
||||||
|
|
||||||
Pull Requests should by default use the `develop` branch as a base. The `master` branch is kept stable and `develop` is periodically merged there. Tags are used for releases. Each Pull Request must be rebased against `develop` and squashed into a single commit, prefixed with the root namespace(s) under `status-im` that they modify. e.g.
|
Pull Requests should by default use the `develop` branch as a base. The `master` branch is kept stable and `develop` is periodically merged there. Tags are used for releases. Each Pull Request must be rebased against `develop` and squashed into a single commit.
|
||||||
> "contacts, ios: add contact stylistic changes"
|
|
||||||
|
|
||||||
|
### Commit messages
|
||||||
|
|
||||||
|
Commit message should help understand why (and what) has been fixed. Add a summary as first line if the message is too long.
|
||||||
|
When fixing bugs the first line should be prefixed with `[FIX #123]`.
|
||||||
|
|
||||||
|
As a good practice message starts with capitalized verb in the imperative mood.
|
||||||
|
|
||||||
### Walkthrough
|
### Walkthrough
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue