mirror of
https://github.com/qTox/qTox.git
synced 2024-03-22 14:00:36 +08:00
Merge branch 'pr2755'
This commit is contained in:
commit
defae04227
37
CONTRIBUTING.md
Normal file
37
CONTRIBUTING.md
Normal file
|
@ -0,0 +1,37 @@
|
||||||
|
# Filing an issue
|
||||||
|
|
||||||
|
### Must read
|
||||||
|
* If you aren't sure, you can ask on the [**IRC channel**](https://webchat.freenode.net/?channels=qtox) or read our [**wiki**](https://github.com/tux3/qTox/wiki) first.
|
||||||
|
* Do a quick **search**. Others might already reported the issue.
|
||||||
|
* Write in **English**!
|
||||||
|
* Provide **version** information: (You can find version numbers at menu `Settings → About`)
|
||||||
|
```
|
||||||
|
qTox:
|
||||||
|
Commit hash:
|
||||||
|
toxcore:
|
||||||
|
Qt:
|
||||||
|
OS version:
|
||||||
|
|
||||||
|
```
|
||||||
|
* Provide **steps** to reproduce the problem, it will be easier to pinpoint the fault.
|
||||||
|
* **Screenshots**! A screenshot is worth a thousand words. just upload it. [(How?)](https://help.github.com/articles/file-attachments-on-issues-and-pull-requests)
|
||||||
|
|
||||||
|
### Good to know
|
||||||
|
* **Patience**. The dev team is small and resource limited. Devs finding their free time, analyzing the problem and fixing the issue, it all takes time. :clock3:
|
||||||
|
* If you can code, why not become a **contributor** by fixing the issue and open a pull request? :wink:
|
||||||
|
* Harsh words or threats won't help your situation. What's worse, your complain will (very likely) to be **ignored**. :fearful:
|
||||||
|
|
||||||
|
|
||||||
|
# Opening a pull request
|
||||||
|
|
||||||
|
### Must read
|
||||||
|
* Read our [**coding guidelines**](https://github.com/tux3/qTox/wiki/Coding).
|
||||||
|
* Keep the title **short** and provide a **clear** description about what your pull request does.
|
||||||
|
* Provide **screenshots** for UI related changes.
|
||||||
|
* Keep your git commit history **clean** and **precise**. Commits like `xxx fixup` should not appear.
|
||||||
|
* If your commit fix a reported issue (for example #4134), add the following message to the commit `Fixes #4134.`. Example [here](https://github.com/tux3/qTox/commit/87160526d5bafcee7869d6741a06045e13d731d5).
|
||||||
|
|
||||||
|
### Good to know
|
||||||
|
* **Search** pull request history! Others might already implemented your idea and is waiting to be merged (or got rejected already). Save your precious time by doing a search first.
|
||||||
|
* When resolving merge conflicts, do `git rebase <target_branch_name>`, don't do `git pull`. Then you can start fixing the conflicts. Here is a good explanation: [link](https://www.atlassian.com/git/tutorials/merging-vs-rebasing).
|
||||||
|
|
Loading…
Reference in New Issue
Block a user