Simplify a great deal the CONTRIBUTING file
This commit is contained in:
parent
2135027040
commit
bd73077504
@ -5,44 +5,33 @@ your contributions.
|
||||
|
||||
### I have a question
|
||||
|
||||
Find us on #thelounge channel on Freenode. You might not get an answer right
|
||||
away, but this channel is filled with nice people who will be happy to help you.
|
||||
- Find us on the Freenode channel `#thelounge`. You might not get an answer
|
||||
right away, but this channel is full of nice people who will be happy to
|
||||
help you.
|
||||
|
||||
### I want to report a bug
|
||||
|
||||
First of all, look at the
|
||||
[open and closed issues](https://github.com/thelounge/lounge/issues?q=is%3Aissue)
|
||||
to see if this was not already discussed before.
|
||||
- Look at the [open and closed
|
||||
issues](https://github.com/thelounge/lounge/issues?q=is%3Aissue) to see if
|
||||
this was not already discussed before. If you can't see any, feel free to
|
||||
[open a new issue](https://github.com/thelounge/lounge/issues/new).
|
||||
|
||||
### I want to contribute to the code
|
||||
|
||||
A good starting point if you want to help us but do not have a clear idea of
|
||||
what you can do specifically is to
|
||||
look at the open issues labeled as [*quick and
|
||||
easy*](https://github.com/thelounge/lounge/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3A%22quick+and+easy%22)
|
||||
or [*help
|
||||
wanted*](https://github.com/thelounge/lounge/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3A%22help+wanted%22).
|
||||
|
||||
Also, make sure that your PRs do not contain unnecessary commits. If you think
|
||||
some of your commits should be merged into a single one, feel free to [squash
|
||||
them](https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History).
|
||||
|
||||
Please [rebase](https://git-scm.com/book/en/v2/Git-Branching-Rebasing) outdated
|
||||
PRs on master to help with the reviews (rebasing is preferred over merging to
|
||||
keep a clean history in a branch/PR).
|
||||
|
||||
Additionally, give extra care to your commit messages, as they will help us
|
||||
review your PRs as well as help other contributors in the future, when exploring
|
||||
the history. The general rules are to [use the imperative present
|
||||
tense](https://git-scm.com/book/ch5-2.html#Commit-Guidelines), to start with a
|
||||
single concise line, followed by a blank line and a more detailed explanation
|
||||
when necessary. Tim Pope wrote an [excellent
|
||||
article](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html)
|
||||
on how one should format their commit messages.
|
||||
|
||||
When you send a PR, expect two different reviews from the project
|
||||
maintainers. You can read more about this in the [maintainers'
|
||||
- Make sure to discuss your ideas with the community in an
|
||||
[issue](https://github.com/thelounge/lounge/issues) or on the IRC channel.
|
||||
- Take a look at the open issues labeled as [`help wanted`](https://github.com/thelounge/lounge/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3A%22help+wanted%22)
|
||||
if you want to help without having a specific idea in mind.
|
||||
- Make sure that your PRs do not contain unnecessary commits or merge commits.
|
||||
[Squash commits](https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History)
|
||||
whenever possible.
|
||||
- [Rebase](https://git-scm.com/book/en/v2/Git-Branching-Rebasing) (instead of
|
||||
merge) outdated PRs on the `master` branch.
|
||||
- Give extra care to your commit messages. Use the [imperative present
|
||||
tense](https://git-scm.com/book/ch5-2.html#Commit-Guidelines) and [follow Tim
|
||||
Pope's guidelines](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).
|
||||
- Each PR will be reviewed by at least two different project maintainers. You
|
||||
can read more about this in the [maintainers'
|
||||
corner](https://github.com/thelounge/lounge/wiki/Maintainers'-corner).
|
||||
|
||||
Please document any relevant changes in the documentation that can be found
|
||||
[in its own repository](https://github.com/thelounge/thelounge.github.io).
|
||||
- Please document any relevant changes in the documentation that can be found
|
||||
[in its own repository](https://github.com/thelounge/thelounge.github.io).
|
||||
|
Loading…
Reference in New Issue
Block a user