How to contribute to DFHack¶
DFHack’s source code is hosted on GitHub. To obtain the code, you do not need an account - see the compilation instructions for details. However, to contribute code to DFHack, you will need a GitHub account to submit pull requests. DFHack consists of several repositories, so you will need to fork the repository (or repositories) containing the code you wish to modify. GitHub has several documentation pages on these topics, including:
Proposing changes with pull requests (note: see Pull request guidelines for some DFHack-specific information)
In general, if you are not sure where or how to make a change, or would like advice before attempting to make a change, please see Getting help for ways to contact maintainers. If you are interested in addressing an issue reported on the issue tracker, you can start a discussion there if you prefer.
The sections below cover some guidelines that contributions should follow:
General contribution guidelines¶
If convenient, compile on multiple platforms when changing anything that compiles. Our CI should catch anything that fails to build, but checking in advance can sometimes let you know of any issues sooner.
Update documentation when applicable - see Documentation standards for details.
docs/about/Authors.rstwhen applicable. See Building the changelogs for more information on the changelog format.
Submit ideas and bug reports as issues on GitHub. Posts in the forum thread or on Disord can easily get missed or forgotten.
Work on reported problems will take priority over ideas or suggestions.
Four space indents for C++. Never use tabs for indentation in any language.
LF (Unix style) line terminators
No trailing whitespace
Opening and closing braces on their own lines or opening brace at the end of the previous line
Braces placed at original indent level if on their own lines
#includedirectives should be sorted: C++ libraries first, then DFHack modules, then
df/headers, then local includes. Within each category they should be sorted alphabetically.
Pull request guidelines¶
Pull requests should be based on (and submitted to) the default branch of the relevant repo, which is the branch you see when you access the repo on GitHub or clone the repo without specifying a branch. As of 0.47.04-r1, this is
developfor the main DFHack repo and
masterfor other repos.
We often leave feedback as comments on pull requests, so be sure that you have notifications turned on or that you check back for feedback periodically.
Use a new branch for each feature or bugfix so that your changes can be merged independently (i.e. not the
developbranch of your fork).
An exception: for a collection of small miscellaneous changes (e.g. structures research), one branch instead of many small branches is fine. It is still preferred that this branch be dedicated to this purpose, i.e. not
develop. Your pull request may be merged at any point unless you indicate that it isn’t ready (see below), but you can continue to push to the same branch and open new pull requests as needed.
Try to keep pull requests relatively small so that they are easier to review and merge.
If you expect to make a large number of related additions or changes (e.g. adding a large new plugin), multiple PRs are preferred, as they allow more frequent (and easier) feedback. If development of this feature is expected to take a while, we may create a dedicated branch to merge your pull requests into instead of the repo’s default branch.
If you plan to make additional changes to your pull request in the near future, or if it isn’t quite ready to be merged, mark it as a draft pull request or add “WIP” to the title. Otherwise, your pull request may be reviewed and/or merged prematurely.
Other ways to help¶
DFHack is a software project, but there’s a lot more to it than programming. If you’re not comfortable programming, you can help by:
reporting bugs and incomplete documentation
improving the documentation (C++ api is rife)
finding third-party scripts to add
writing tutorials for newbies
All those things are crucial, and often under-represented. So if that’s your thing, go get started!