2015-02-20 15:22:44 +01:00
|
|
|
# Contributing
|
2014-05-26 21:26:52 +02:00
|
|
|
|
|
|
|
|
|
|
|
## Bugs
|
2014-05-24 22:13:42 +02:00
|
|
|
|
2015-02-20 15:22:44 +01:00
|
|
|
Please look at `https://github.com/awesomeWM/awesome/issues`.
|
2014-05-24 22:13:42 +02:00
|
|
|
|
2014-05-26 21:26:52 +02:00
|
|
|
## Style
|
2014-05-24 22:13:42 +02:00
|
|
|
|
|
|
|
If you intend to patch and contribute to awesome, please respect the
|
|
|
|
following guidelines.
|
|
|
|
|
|
|
|
Imitate the existing code style. For concrete rules:
|
|
|
|
|
|
|
|
- Use 4 spaces indentation, do not use tabulator characters;
|
|
|
|
|
|
|
|
- Place braces alone on new lines, and do not place braces for single
|
2015-02-26 12:08:53 +01:00
|
|
|
line statement where it is not needed, i.e. no:
|
2014-05-24 22:13:42 +02:00
|
|
|
|
|
|
|
if(bla) {
|
|
|
|
x = 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
- Do not put a space after if, for, while or function call statements;
|
|
|
|
|
|
|
|
- The preferred line length is 80 characters;
|
|
|
|
|
|
|
|
- Use `/* */` for comments;
|
|
|
|
|
|
|
|
- Use the API: there's a list of `a_*()` function you should use instead
|
|
|
|
of the standard libc ones. There is also common API for linked list,
|
2015-02-26 12:08:53 +01:00
|
|
|
tabulars, etc.;
|
2014-05-24 22:13:42 +02:00
|
|
|
|
|
|
|
- Be *clear* in what you do;
|
|
|
|
|
|
|
|
- Prefix your function name with the module they are enhancing,
|
|
|
|
i.e. if you add a function to manipulate a tag prefix it with `tag_`.
|
|
|
|
|
|
|
|
- Write documentation for any new functions, options, whatever.
|
|
|
|
|
|
|
|
A vim modeline is set in each file to respect this.
|
|
|
|
|
2015-02-26 12:09:17 +01:00
|
|
|
### Documentation of lua files
|
|
|
|
|
|
|
|
For documentation purposes LDoc---see
|
|
|
|
[here](https://stevedonovan.github.io/ldoc/manual/doc.md.html) for its
|
|
|
|
documentation---is used. Comments, that shall be parsed by LDoc have the following format:
|
|
|
|
|
|
|
|
--- summary.
|
|
|
|
-- Description; this can extend over
|
|
|
|
-- several lines
|
|
|
|
|
|
|
|
-----------------
|
|
|
|
-- This will also do.
|
|
|
|
|
|
|
|
--[[--
|
|
|
|
Summary. A description
|
|
|
|
...;
|
|
|
|
]]
|
|
|
|
|
|
|
|
One can use the full power of [Markdown](http://daringfireball.net/projects/markdown)
|
|
|
|
with the extensions of [Discount](http://www.pell.portland.or.us/~orc/Code/discount/)
|
|
|
|
for mark-up in the comments.
|
|
|
|
|
|
|
|
Every module and class should have a short description at its beginning which
|
|
|
|
should include `@author author`, `@copyright year author`,
|
|
|
|
`@release @AWESOME_VERSION@` and `@module module-name` or `@classmod class-name`.
|
|
|
|
Parameters of functions should be documented by `@param parameter description`,
|
|
|
|
return values via `@return description` or `@treturn type description`. For a
|
|
|
|
more comprehensive description of the available tags see the
|
|
|
|
[LDoc documentation](https://stevedonovan.github.io/ldoc/manual/doc.md.html).
|
|
|
|
|
|
|
|
In addition to the regular tags provided by LDoc there are also some aliases for
|
|
|
|
tags defined in `docs/config.ld`.
|
|
|
|
|
2014-05-26 21:26:52 +02:00
|
|
|
## Patches
|
2014-05-24 22:13:42 +02:00
|
|
|
|
|
|
|
If you plan to submit patches, you should follow the following guidelines.
|
|
|
|
|
2014-05-26 21:26:52 +02:00
|
|
|
### Commits
|
2014-05-24 22:13:42 +02:00
|
|
|
|
|
|
|
- make commits of logical units;
|
|
|
|
- do not modify piece of code not related to your commit;
|
|
|
|
- do not try to fix style of code you are not writing,
|
|
|
|
it's just adding noise for no gain;
|
2015-02-26 12:08:53 +01:00
|
|
|
- check for unnecessary whitespace with `git diff --check` before committing;
|
2014-05-24 22:13:42 +02:00
|
|
|
- do not check in commented out code or unneeded files;
|
|
|
|
- provide a meaningful commit message;
|
|
|
|
- the first line of the commit message should be a short;
|
|
|
|
description and should skip the full stop;
|
|
|
|
- if you want your work included, add a
|
2015-02-20 16:57:34 +01:00
|
|
|
`Signed-off-by: Your Name <you@example.com>` line to the
|
2015-02-26 12:08:53 +01:00
|
|
|
commit message (or just use the option `-s` when committing);
|
2014-05-24 22:13:42 +02:00
|
|
|
- make sure that you have tests for the bug you are fixing.
|
2015-02-20 15:22:44 +01:00
|
|
|
- if possible, add a unit test to the test suite under spec/.
|
2014-05-24 22:13:42 +02:00
|
|
|
|
2014-05-26 21:26:52 +02:00
|
|
|
### Patches
|
2014-05-24 22:13:42 +02:00
|
|
|
|
|
|
|
- use `git format-patch -M` to create the patch;
|
|
|
|
- do *not* PGP sign your patch;
|
|
|
|
- be *careful* doing cut & paste into your mailer, not to corrupt whitespaces;
|
|
|
|
- if you change, add or remove the user API, the associated documentation
|
|
|
|
should be updated as well;
|
|
|
|
- send the patch to the list (`awesome-devel@naquadah.org`) if (and only if)
|
|
|
|
the patch is ready for inclusion. If you use git-send-email(1), please
|
|
|
|
*test* it first by sending email to yourself.
|