yquake2remaster/doc/080_contributing.md

33 lines
1.6 KiB
Markdown
Raw Permalink Normal View History

# 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,
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
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.
* 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.
* 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
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.