Contributing
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
Report bugs at https://gitlab.com/ethz_hvl/hvl_ccb/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.
Fix Bugs
Look through the GitLab issues for bugs. Anything tagged with “bug” and “help wanted” is open to whoever wants to implement it.
Implement Features
Look through the GitLab issues for features. Anything tagged with “enhancement” and “help wanted” is open to whoever wants to implement it.
Write Documentation
HVL Common Code Base could always use more documentation, whether as part of the official HVL Common Code Base docs, in docstrings, or even on the web in blog posts, articles, and such.
Submit Feedback
The best way to send feedback is to file an issue at https://gitlab.com/ethz_hvl/hvl_ccb/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.
Remember that this is a volunteer-driven project, and that contributions are welcome :)
Get Started!
Ready to contribute? Here’s how to set up hvl_ccb for local development.
Clone hvl_ccb repo from GitLab.
$ git clone git@gitlab.com:ethz_hvl/hvl_ccb.git
Install your local copy into a virtualenv. Assuming you have virtualenvwrapper installed, this is how you set up your fork for local development:
$ mkvirtualenv hvl_ccb $ cd hvl_ccb/ $ pip install -e .[all] $ pip install -r requirements_dev.txt
Create a branch for local development:
$ git checkout -b name-of-your-bugfix-or-feature
Now you can make your changes locally.
When you’re done making changes, check that your changes pass flake8 and the tests, including testing other Python versions with tox:
$ flake8 hvl_ccb tests $ python setup.py test or py.test $ tox
To get flake8 and tox, just pip install them into your virtualenv. You can also use the provided make-like shell script to run flake8 and tests:
$ ./make.sh lint $ ./make.sh test
Commit your changes and push your branch to GitLab:
$ git add . $ git commit -m "Your detailed description of your changes." $ git push origin name-of-your-bugfix-or-feature
Submit a merge request through the GitLab website.
Merge Request Guidelines
Before you submit a merge request, check that it meets these guidelines:
The merge request should include tests.
If the merge request adds functionality, the docs should be updated. Put your new functionality into a function with a docstring, and add the feature to the list in README.rst.
The merge request should work for Python 3.7. Check https://gitlab.com/ethz_hvl/hvl_ccb/merge_requests and make sure that the tests pass for all supported Python versions.
Tips
To run tests from a single file:
$ py.test tests/test_hvl_ccb.py
or a single test function:
$ py.test tests/test_hvl_ccb.py::test_command_line_interface
If your tests are slow, profile them using the pytest-profiling plugin:
$ py.test tests/test_hvl_ccb.py --profile
or for a graphical overview (you need a SVG image viewer):
$ py.test tests/test_hvl_ccb.py --profile-svg $ open prof/combined.svg
To add dependency, edit appropriate
*requirements
variable in thesetup.py
file and re-run:$ python setup.py develop
To generate a PDF version of the Sphinx documentation instead of HTML use:
$ rm -rf docs/hvl_ccb.rst docs/modules.rst docs/_build && sphinx-apidoc -o docs/hvl_ccb && python -msphinx -M latexpdf docs/ docs/_build
This command can also be run through the make-like shell script:
$ ./make.sh docs-pdf
This requires a local installation of a LaTeX distribution, e.g. MikTeX.
Deploying
A reminder for the maintainers on how to deploy. Create release-N.M.K
branch.
Make sure all your changes are committed. Update or create entry in HISTORY.rst
file, and, if applicable, update AUTHORS.rst
file, update features tables in
README.rst
file, and update API docs:
$ make docs
Commit all of the above, except for the docs/hvl_ccb.dev.picotech_pt104.rst
,
and then run:
$ bumpversion patch # possible: major / minor / patch
$ git push
$ git push --tags
Go to https://readthedocs.org/projects/hvl-ccb/builds/ and check if RTD docs build for the pushed tag passed.
Wait for the CI pipeline to finish successfully; afterwards, run a release check:
$ make release-check
Finally, prepare and push a release:
$ make release
Merge the release branch into master and devel branches with --no-ff
flag and
delete the release branch:
$ git switch master
$ git merge --no-ff release-N.M.K
$ git push
$ git switch devel
$ git merge --no-ff release-N.M.K
$ git push
$ git push --delete origin release-N.M.K
$ git branch --delete release-N.M.K
Finally, prepare GitLab release and cleanup the corresponding milestone:
go to https://gitlab.com/ethz_hvl/hvl_ccb/-/tags/, select the latest release tag, press “Edit release notes” and add the release notes (copy a corresponding entry from
HISTORY.rst
file with formatting adjusted from ReStructuredText to Markdown); press “Save changes”;go to https://gitlab.com/ethz_hvl/hvl_ccb/-/releases, select the latest release, press “Edit this release” and under “Milestones” select the corresponding milestone; press “Save changes”;
go to https://gitlab.com/ethz_hvl/hvl_ccb/-/milestones, make sure that it is 100% complete (otherwise, create a next patch-level milestone and assign it to the ongoing Issues and Merge Requests therein); press “Close Milestone”.