Contributions are welcome, and they are greatly appreciated! Every little bit helps, and credit will always be given.
You can contribute in many ways:
Types of Contributions¶
Report bugs at https://github.com/twosigma/marbles/issues.
If you are reporting a bug, please include:
- Your operating system name and version.
- Any details about your local setup that might be helpful in troubleshooting.
- Detailed steps to reproduce the bug.
The issue template will tell you how to collect version information.
Look through the GitHub issues for bugs. Anything tagged with “bug” and “help wanted” is open to whoever wants to implement it.
Look through the GitHub issues for features. Anything tagged with “enhancement” and “help wanted” is open to whoever wants to implement it.
marbles could always use more documentation, whether as part of the official marbles docs, in docstrings, or even on the web in blog posts, articles, and such.
The best way to send feedback is to file an issue at https://github.com/twosigma/marbles/issues.
If you are proposing a feature:
- Explain in detail how it would work.
- Keep the scope as narrow as possible, to make it easier to implement.
Build a Plugin¶
We’ve developed marbles in a pluggable way, so you can contribute to the marbles ecosystem without committing code to the marbles repo!
Just as marbles provides some mixins you can add to a
by adding a superclass to your test, you can develop more mixins outside the
marbles repo and publish them yourself, they’ll interoperate just fine. Of
course, we’d like to include mixins which may be useful to many people in the
marbles.mixins package, so if you think that’s the case, please send us a
The marbles annotation logging mechanism right now just writes JSON structured
data to a file. You can use
Spark, or any
other tool that understands JSON to store and analyze them after the fact. But,
you can also implement the marbles logging interface to do something else with
the assertion metadata, instead of logging it to disk. If you’d like to share
your logging plugin, or discuss ideas for how to build one, just open an issue
and we can discuss it with you there.
Ready to contribute? Here’s how to set up marbles for local development.
Fork the marbles repo on GitHub here.
Clone your fork locally:
$ git clone firstname.lastname@example.org:your_name_here/marbles.git
Install your local copy into a virtualenv. If you have pipenv installed, you can run:
$ cd marbles/ $ pipenv install --dev $ pipenv shell
This will install all of the
marblesdevelopment dependencies, install
marblesin development mode, so your changes to the files in your clone will take effect immediately, and put you in a shell where you can run the tests, build the docs, etc.
If you don’t use pipenv, you can get the same effect inside any other virtualenv by running:
$ pip install -r requirements.txt
Create a branch for local development:
$ git checkout -b name-of-your-bugfix-or-feature-or-docs
Now you can make your changes locally.
marblesis developed as separate packages in the namespace package
As you make changes, you can run the tests and lint with flake8. These should be run inside the package you’ve made changes to, so if you’ve made changes to
marbles.core, you should run this:
$ cd marbles/core $ python setup.py flake8 $ python setup.py test
Don’t worry about bumping version numbers yourself. We’ll handle this in the release that includes your changes.
For more developer workflows (linting, testing, test coverage, docs), see Maintainer’s Guide.
Commit your changes and push your branch to GitHub:
$ git add . $ git commit -m "Your detailed description of your changes." $ git push origin name-of-your-bugfix-or-feature
Submit a pull request through the GitHub website.
We’ll review your changes, merge them, and include them in the next release.
Pull Request Guidelines¶
Before you submit a pull request, check that it meets these guidelines:
- The pull request should include tests.
- If the pull request adds functionality, the docs should be updated. Make sure your new functionality is documented with docstrings and appropriate additions to the Sphinx docs, and add the feature to the list in README.md.
- The pull request should work for Python 3.5, 3.6, and 3.7. Check https://travis-ci.org/twosigma/marbles/pull_requests and make sure that the tests pass for all supported Python versions.
- In order to accept your code contributions, please fill out the appropriate Contributor License Agreement in the cla folder and submit it to email@example.com. We need this before we can accept your pull request.