Skip to content

Latest commit

 

History

History
216 lines (146 loc) · 6.98 KB

CONTRIBUTING.rst

File metadata and controls

216 lines (146 loc) · 6.98 KB

Contributing

There are several strategies on how to contribute to a project on github. Here, I explain the one I use for all the project I am participating. You can use this same strategy to contribute to this template or to suggest contributions to your project.

Fork this repository

Fork this repository before contributing. It is a better practice, possibly even enforced, that only pull request from forks are accepted. In my opinion enforcing forks creates a cleaner representation of the contributions to the project.

Clone the main repository

Next, clone the main repository to your local machine:

git clone https://github.com/joaomcteixeira/python-project-skeleton.git
cd python-project-skeleton

Add your fork as an upstream repository:

git remote add myfork git://github.com/YOUR-USERNAME/python-project-skeleton.git
git fetch myfork

Install for developers

Create a dedicated Python environment where to develop the project.

If you are using pip follow the official instructions on Installing packages using pip and virtual environments, most likely what you want is:

python3 -m venv newenv
source newenv/bin/activate

If you are using Anaconda go for:

conda create --name newenv python=3.7
conda activate newenv

Where newenv is the name you wish to give to the environment dedicated to this project.

Either under pip or conda, install the package in develop mode. Install also :ref:`tox<Uniformed Tests with tox>`.

python setup.py develop
pip install tox

This configuration, together with the use of the src folder layer, guarantees that you will always run the code after installation. Also, thanks to the develop flag, any changes in the code will be automatically reflected in the installed version.

Make a new branch

From the main branch create a new branch where to develop the new code.

git checkout main
git checkout -b new_branch

Note the main branch is from the main repository.

Develop the feature and keep regular pushes to your fork with comprehensible commit messages.

git status
git add (the files you want)
git commit (add a nice commit message)
git push myfork new_branch

While you are developing, you can execute tox as needed to run your unit tests or inspect lint, or other integration tests. See the last section of this page.

Update your branch

It is common that you need to keep your branch update to the latest version in the main branch. For that:

git checkout main  # return to the main branch
git pull  # retrieve the latest source from the main repository
git checkout new_branch  # return to your devel branch
git merge --no-ff main  # merge the new code to your branch

At this point you may need to solve merge conflicts if they exist. If you don't know how to do this, I suggest you start by reading the official docs

You can push to your fork now if you wish:

git push myfork new_branch

And, continue doing your developments are previously discussed.

Update CHANGELOG

Update the changelog file under CHANGELOG.rst with an explanatory bullet list of your contribution. Add that list right after the main title and before the last version subtitle:

Changelog
=========

* here goes my new additions
* explain them shortly and well

vX.X.X (1900-01-01)
-------------------

Also add your name to the authors list at docs/AUTHORS.rst.

Pull Request

Once you finished, you can create a pull request to the main repository, and engage with the community.

Before submitting a Pull Request, verify your development branch passes all tests as :ref:`described below <Uniformed Tests with tox>` . If you are developing new code you should also implement new test cases.

Uniformed Tests with tox

Thanks to Tox we can have a unified testing platform that runs all tests in controlled environments and that is reproducible for all developers. In other words, it is a way to welcome (force) all developers to follow the same rules.

The tox testing setup is defined in a configuration file, the tox.ini, which contains all the operations that are performed during the test phase. Therefore, to run the unified test suite, developers just need to execute tox, provided tox is installed in the Python environment in use.

pip install tox
# or
conda install tox -c conda-forge

One of the greatest advantages of using tox together with the :ref:`src layout <The src layout>` is that unit test actually perform on the installed source (our package) inside an isolated deployment environment. In order words, tests are performed in an environment simulating a post-installation state instead of a pre-deploy/development environment. Under this setup, there is no need, in general cases, to distribute unit test scripts along with the actual source, in my honest opinion - see MANIFEST.in.

Before creating a Pull Request from your branch, certify that all the tests pass correctly by running:

tox

These are exactly the same tests that will be performed online in the Github Actions.

Also, you can run individual testing environments if you wish to test only specific functionalities, for example:

tox -e lint  # code style
tox -e build  # packaging
tox -e docs  # only builds the documentation
tox -e test  # runs unit tests