These are loose checklist for contributors to follow when submitting patches.
-
Small commits
You can contribute by a lot of ways from improving README, fixing typos, coding style, specific bugs, performance optimizations. However, it is preferred that you break up your commits to single logical change using
git add -p
so it is easier to review the patch. The larger the change, the more necessary it is for the commit to be broken up to tiny little pieces. If your change is large but consistent throughout (e.g. fixing a specific coding style that happens on almost every file), that can be counted as single logical change. -
File and directory-based commit message
We're starting to use commit messages that looks like this:
sdl: fixed some typos in render.go
orexamples/render_goroutine: fixed a dereferenced nil pointer
where it starts with folder hierarchy. It's not something strictly required but we would prefer it to be followed. -
Compatibility with SDL 2.0.0
The binding should compile with the oldest version SDL2. If there's a function added to the binding but is not supported by the older SDL2 version, a stub function must be provided. See
sdl/filesystem.go
for an example.
You can start by forking the repository, modify the fork, push the change to your fork, and then send pull requests. Here are some instructions on how you can work with your own fork.