Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revisionNext revisionBoth sides next revision | ||
dev:howto:contribute [2013/02/02 15:54] – jannis | dev:howto:contribute [2013/02/02 16:06] – [Purpose of this document] jannis | ||
---|---|---|---|
Line 3: | Line 3: | ||
===== Purpose of this document ===== | ===== Purpose of this document ===== | ||
- | At the time of writing (FOSDEM 2013), the entry barrier for contributing to Xfce is substantially higher than it could be. There are several hurdles that people have to figure out themselves. There is no easily accessible list of low-hanging fruit or tasks to get started with. Also, there is no defined, transparent contribution process. Our aim should be to make obvious to interested developers how they can contribute and what they can contribute. | + | At the time of writing (FOSDEM 2013), the entry barrier for contributing to Xfce is substantially higher than it could be. There are several hurdles that people have to figure out themselves. There is, for example, |
- | The purpose of this document is to come up with a concept for providing low-hanging fruit to the developer community and a contribution process that removes uncertainty about what the right way/format of contributing code is. This document is only about contributing code since other areas, like [[http:// | + | The purpose of this document is to come up with a concept for providing low-hanging fruit to the developer community and a contribution process that removes uncertainty about what the right way/format of contributing code is. |
+ | |||
+ | This document is only about contributing code since other areas, like [[http:// | ||
===== Present State ===== | ===== Present State ===== | ||
Line 46: | Line 48: | ||
* Make sure to pull in the latest Xfce master frequently. | * Make sure to pull in the latest Xfce master frequently. | ||
* Keep commits clean: | * Keep commits clean: | ||
- | * Make incremental, | + | * Make incremental, |
- | * Keep code working after every commit | + | * Keep code working after every commit. |
- | * Comment the code you write | + | * Comment the code you write. |
- | * Write commit messages using the standard Git message format | + | * Write commit messages using the standard Git message format. |
- | * Don't fear the rebase (against the Xfce master branch) | + | * Don't fear the rebase (against the Xfce master branch). |
=== 3. Push === | === 3. Push === | ||
Line 75: | Line 77: | ||
Fill out the SUBJECT and MESSAGE by briefly describing what you are submitting. Then save the file and close the editor. '' | Fill out the SUBJECT and MESSAGE by briefly describing what you are submitting. Then save the file and close the editor. '' | ||
- | == Configuring '' | + | **Configuring '' |
You will need to configure a number of items for '' | You will need to configure a number of items for '' | ||
Line 100: | Line 102: | ||
* Testing the changes. | * Testing the changes. | ||
* Comments on: | * Comments on: | ||
- | * Coding style and bad code design | + | * Coding style and bad code design. |
- | * Potential issues | + | * Potential issues. |
- | * Possible side-effects | + | * Possible side-effects. |
The reviews will be sent as replies to your cover letter and patch emails. Thanks to having the patches included in your submission as plain text, they can be commented on in-line. | The reviews will be sent as replies to your cover letter and patch emails. Thanks to having the patches included in your submission as plain text, they can be commented on in-line. |