Contributor Guide

If you don't have a lot of experience with submitting patches or PRs (pull requests) to an OpenSource project please start by reading the Contributing to Open Source Projects document. (It's probably a good idea to review the document even if you do have experience, just to make sure that you're "on the same page".) The document provides a clear and concise set of general guidelines for contributing to projects which will help streamline the process for both you and the project maintainers.

The rest of this page will simply emphasize a few of the important things, building upon the guidelines in the Contributing... document.

  1. If your contribution is intended to fix a bug or other issue, make sure that there is a GitHub issue created for it. If there isn't, then [create one](/pages/reporting-issues/). This is not necessary for enhancements or new features.

  2. If your contribution is large or complex it may be a good idea to discuss it first in the GitHub issue or on wxPython-dev. It may save you some work and frustration if your contribution may need to be implemented differently than what you had in mind, or etc.

  3. Always create a new branch for your contribution, do not use your master branch as the source of the PR. Why? Two reasons: Because everybody has a branch named master, so giving your contribution a different name eliminates that particular confusion. Secondly, if you use your master branch then that means that you should not make any other unrelated changes to your master branch until after your PR has been merged.

    For example, to create a branch for a feature that "foos the bars":

    <Do a fork and clone of Phoenix if you haven't already...>
    $ git checkout master
    $ git pull
    $ git checkout -b foo-the-bars
    <hack hack hack>
    $ git push origin HEAD
    <submit PR based on your 'foo-the-bars' branch from your GitHub fork>
  4. If there is an issue associated with your contribution, then include the issue number in your PR's description text, (for example, fixes #1234). This will cause the issue to be automatically closed when the PR is merged.

  5. Do not put more than one fix/feature in the same branch or PR. Make a new distinct branch and PR for each of them.

  6. Test. Test. Test.