diff options
author | Lukas Fleischer <lfleischer@archlinux.org> | 2015-07-22 13:49:51 +0200 |
---|---|---|
committer | Lukas Fleischer <lfleischer@archlinux.org> | 2015-08-08 12:59:24 +0200 |
commit | ab2577525970e415f9ad2439ae1bb2c51f479c8a (patch) | |
tree | b83526e0fea39f5f528bfe9d7a39c3f25e84d489 /doc/CodingGuidelines | |
parent | 080b6f3d123c465f70a8f9f1e724111dd1b3cf56 (diff) | |
download | aurweb-ab2577525970e415f9ad2439ae1bb2c51f479c8a.tar.xz |
Move documentation to a subdirectory
Create a new subdirectory doc/ that contains documentation.
Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
Diffstat (limited to 'doc/CodingGuidelines')
-rw-r--r-- | doc/CodingGuidelines | 54 |
1 files changed, 54 insertions, 0 deletions
diff --git a/doc/CodingGuidelines b/doc/CodingGuidelines new file mode 100644 index 0000000..46537bb --- /dev/null +++ b/doc/CodingGuidelines @@ -0,0 +1,54 @@ +Coding Guidelines +================= + +DISCLAIMER: We realise the code doesn't necessarily follow all the rules. +This is an attempt to establish a standard coding style for future +development. + +Coding style +------------ + +Column width: 79 columns or less within reason. + +Indentation: tabs (standard eight column width) +Please don't add any mode lines. Adjust your editor to display tabs to your +preferred width. Generally code should work with the standard eight column +tabs. + +No short open tags. '<?' +Do not end files with a close tag. '?>' +Try embedding as little XHTML in the PHP as possible. +Consider creating templates for XHTML. + +All markup should conform to XHTML 1.0 Strict requirements. +You can use http://validator.w3.org to check the markup. + +Prevent PHP Notices by using isset() or empty() in conditionals that +reference $_GET, $_POST, or $_REQUEST variables. + +MySQL queries should generally go into functions. + +Submitting patches +------------------ + +!!! PLEASE TEST YOUR PATCHES BEFORE SUBMITTING !!! +Submit uncompressed git-formatted patches to aur-dev@archlinux.org. + +You will need to register on the mailing list before submitting: +https://mailman.archlinux.org/mailman/listinfo/aur-dev + +Base your patches on the master branch as forward development is done there. +When writing patches please keep unnecessary changes to a minimum. + +Try to keep your commits small and focused. +Smaller patches are much easier to review and have a better chance of being +pushed more quickly into the main repo. Smaller commits also makes reviewing +the commit history and tracking down specific changes much easier. + +Try to make your commit messages brief but descriptive. + +Glossary +-------- +git-formatted patch: + A patch that is produced via `git format-patch` and is sent via + `git send-email` or as an inline attachment of an email. |