cadey
/
elemental-ircd
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Update contributing guidelines

This commit is contained in:
Sam Dodrill 2015-02-20 00:11:51 -08:00
parent 0361eeeedf
commit cc0c8d338d
1 changed files with 31 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
CONTRIBUTING.md
View File

@ -4,10 +4,8 @@ Please open contributions as either issue reports or pull requests. We will ask
that anything that requires testing outside our normally supported platforms be that anything that requires testing outside our normally supported platforms be
marked as such. marked as such.
As ircd is such an ancient project with varying code styles, please try to Style
follow the current coding style of the file you are in. If all else fails, -----
please use the [Linux Kernel](https://www.kernel.org/doc/Documentation/CodingStyle)
coding style.
Please run all code against the following `astyle` command before sending in Please run all code against the following `astyle` command before sending in
a pull request: a pull request:
@ -16,5 +14,30 @@ a pull request:
$ astyle --style=linux --mode=c -n $file $ astyle --style=linux --mode=c -n $file
``` ```
It should be trivial to set up your text editor to do this for you. It should be trivial to set up your text editor to do this for you. If you use
vim, add this to your vim configuration:
```vimscript
" Lvimrc
" if .lvimrc exists in current or parent directory of the currently loaded file,
" load it as config
if filereadable('../.lvimrc')
source ../.lvimrc
endif
if filereadable('./.lvimrc')
source ./.lvimrc
endif
```
Otherwise please make sure the appropriate command is ran as part of your
editing process before you send a pull request. All pull requests that do not
follow the coding style will not be considered until they follow the coding
style.
Testing
-------
When testing Elemental, please be sure to test it on a network of at least
3 instances of elemental. Please also be sure to have one of these instances
run **without** the patch you are testing, to be able to show that there is
a difference.