2018-11-07 15:28:19 +00:00
|
|
|
# Contributing Code
|
|
|
|
|
|
|
|
You want to contribute code to Yamagi Quake II? That's good! We're
|
|
|
|
always interested in contributions, as long as they're in the scope of
|
2018-11-07 21:35:58 +00:00
|
|
|
our little project. It doesn't matter if you're sending new features,
|
2018-11-07 15:28:19 +00:00
|
|
|
bugfixes or even documentation updates.
|
|
|
|
|
|
|
|
As a general note: **Ask before writing code!** Nobody was ever hurt for
|
|
|
|
asking if his or her idea is in scope for Yamagi Quake II. And an early
|
|
|
|
"no" is always better then rejections after having put tens of hours
|
|
|
|
into the idea.
|
|
|
|
|
|
|
|
Some rules to follow:
|
|
|
|
|
2018-11-07 21:35:58 +00:00
|
|
|
* Use GitHub! Sign up for an account, fork our repository and send a
|
2018-11-07 15:28:19 +00:00
|
|
|
pull request after you're satisfied with your work. We won't accept
|
2018-11-07 21:35:58 +00:00
|
|
|
patches sent by email or - even worse - as pastebin links.
|
2018-11-07 15:28:19 +00:00
|
|
|
* Create a history of small, distinct commits. Several small commits are
|
|
|
|
always better then one big commit. They make your changes more
|
|
|
|
understandable and ease debugging.
|
|
|
|
* Never ever add new dependencies. We won't accept code that add new
|
|
|
|
dependencies. If in doubt, because you really need that nice little
|
|
|
|
library, ask.
|
|
|
|
* Make sure that your code is compiling without warnings with at least
|
2018-11-07 21:35:58 +00:00
|
|
|
current versions of GCC and Clang. Also make sure that it's working on
|
|
|
|
both unixoid platforms and Windows.
|
2018-11-07 15:28:19 +00:00
|
|
|
* Don't do unnecessary cleanups. Yes, your linter or sanity checker may
|
2018-11-07 21:35:58 +00:00
|
|
|
complain. But that's your problem and not ours. Cleanups often bring
|
2018-11-07 15:28:19 +00:00
|
|
|
next to no advantage, Quake II has always been a mess and it'll stay a
|
|
|
|
mess until the sun collapses. And cleanups are hard to test, often
|
|
|
|
introduce new bugs and make debugging harder.
|
|
|
|
* Stick to the code style of the file you're editing.
|