Skip to content

Latest commit

 

History

History
131 lines (97 loc) · 5.87 KB

CONTRIBUTING.md

File metadata and controls

131 lines (97 loc) · 5.87 KB
Raw

Contributing

Galaxy welcomes new development! This document briefly describes how to contribute to the core galaxy repository. For information on contributing more broadly to the Galaxy ecosystem and a deeper discussion of some of these points - please see the Develop section of the Galaxy Community Hub and the Galaxy Code Architecture slides that are part of the Galaxy Training Materials.

Before you Begin

If you have an idea for a feature to add or an approach for a bugfix, it is best to communicate with Galaxy developers early. The primary venue for this is the GitHub issue tracker. Browse through existing GitHub issues and if one seems related, comment on it. For more direct communication, Galaxy developers are generally available via Gitter and on the development mailing list.

If you're looking to help but aren't sure where to start, we also maintain a tag on GitHub for smaller issues we believe would make the best entry points for new developers.

Reporting a new issue

If no existing Galaxy issue seems appropriate, a new issue can be opened using this form.

How to Contribute

All changes to the core galaxy repository should be made through pull requests (with just two exceptions outlined below).

If you are new to Git, the Software Carpentry's Version Control with Git tutorial is a good place to start. More learning resources are listed at https://help.github.com/en/github/getting-started-with-github/git-and-github-learning-resources

  1. Make sure you have a free GitHub account.

  2. Fork the galaxy repository on GitHub to make your changes. To keep your copy up to date with respect to the main repository, you need to frequently sync your fork:

    $ git remote add upstream https://github.com/galaxyproject/galaxy
$ git fetch upstream
$ git checkout dev
$ git merge upstream/dev

  3. Choose the correct branch to develop your changes against.

    • The master branch is kept in sync with the latest tagged release, but should not be used as the base (i.e. target) branch of a pull request.

    • Additions of new features to the codebase should be based off the dev branch (git checkout -b feature_branch dev), with few exceptions.

    • Most bug fixes should target the oldest supported release exhibiting the issue (git checkout -b bugfix_branch release_XX.XX).

    • Serious security problems should not be fixed via pull request - please see the Galaxy security policies for information about responsibly disclosing security issues.

  4. If your changes modify code please ensure the resulting files conform to the style guidelines below.

    If you are working on the Galaxy user interface (i.e. JavaScript, styles, etc.), see more information in the client README.

  5. Galaxy contains hundreds of tests of different types and complexity and running each is difficult and probably not reasonable on your workstation. So please review the running tests documentation and run any that seem relevant.

    If possible, also try to add new tests for the features added or bugs fixed by your pull request.

    Developers reviewing your pull request will be happy to help you add or run the relevant tests as part of the pull request review process.

  6. Commit and push your changes to your fork.

  7. Open a pull request with these changes. Your pull request message ideally should include:

    • Why you made the changes (e.g. references to GitHub issues being fixed).

    • A description of the implementation of the changes.

    • How to test the changes, if you haven't included specific tests already.

  8. The pull request should pass all the continuous integration tests which are automatically started by GitHub using e.g. Travis CI.

  9. Your pull request will be handled according to some rules.

Style guidelines

Python

  • Galaxy follows PEP-8, with particular emphasis on readability being the ultimate goal:
    • 4 spaces (not tabs!) per indentation level
    • divergences from PEP-8 are listed in the [flake8] section of the setup.cfg file.
  • Python imports should be ordered following the smarkets style.
  • Python docstrings need to be in reStructured Text (RST) format and compatible with Sphinx.

A Quick Note about Tools

For the most part, Galaxy tools should be published to a Tool Shed and not in this repository directly. More information about tool development can be found on the community hub.