Skip to content

Latest commit

 

History

History
100 lines (64 loc) · 4.22 KB

CONTRIBUTING.md

File metadata and controls

100 lines (64 loc) · 4.22 KB
Raw

Contribution Guidelines

Here is a simple guideline to get you started with your first contribution.

  1. Set up your environment to follow our formatting guidelines and to use signed-commits.
  2. Use issues to discuss the suggested changes. Create an issue describing changes if necessary and add labels to ease orientation.
  3. Fork data-gradients so you can make local changes and test them.
  4. Create a new branch for the issue. The branch naming convention is enforced by the CI/CD so please make sure you are using feature/DG-*** or hotfix/DG-*** format otherwise it will fail.
  5. Implement your changes along with relevant tests for the issue. Please make sure you are covering unit-tests where required.
  6. Create a pull request against master branch.

Code Style

We follow the reStructuredText docstring format (default of PyCharm), along with typing.

def python_function(first_argument: int, second_argument: int) -> str:
    """Do something with the two arguments.

    :param first_argument: First argument to the function
    :param second_argument: Second argument to the function
    :return: Description of the output
    """

Code Formatting

We enforce black code formatting in addition to existing flake8 checks.

To ensure everyone uses same code style, a project-wise configuration file has been added to DG repo. It ensures all formatting will be exactly the same regardless of OS, python version or the place where code formatting check is happening.

Installation

To start, one need to install required development dependencies (actual versions of black, flake8 and git commit hooks):

$ pip install -r requirements.dev.txt

Pre-Commit Hooks

A pre-commit hook as an easy way to ensure all files in the commit are already formatted accordingly and pass linter checks. If they are not, the git will prevent commit of problematic files unless errors are fixed.

To start, run the following command from DG repo root:

$ pip install pre-commit
$ pre-commit install

The command should complete without errors. Once done, all your upcoming commits will be checked via black & flake8.

Usage

Just run $ black . from the DG root. It will reformat the whole repo. For flake8: $ flake8 --statistics --config scripts/flake8-config setup.py .

Signed Commits

Background

Signed commits provide a way to verify the authenticity and integrity of the code changes made by a particular developer, as the commit is cryptographically signed using their private GPG key. This helps ensure that the code changes were made by the intended person and have not been tampered with during transit.

You can find more information here.

Add GPG key to GitHub

  1. Generate a new GPG key
  2. Copy the GPG key by running the command on step 12 from the link above
$ gpg --armor --export 3AA5C34371567BD2
  1. Add the new GPG key to your GitHub account

Use GPG key

$ git config --global user.signingkey 3AA5C34371567BD2
$ git config --global gpg.program /usr/local/bin/gpg
$ git config --global commit.gpgsign true

Jupyter Notebooks Contribution

Pulling updates from remote might cause merge conflicts with jupyter notebooks. The tool nbdime might solve this.

  • Installing nbdime
pip install ndime
  • Run a diff between two notebooks
nbdiff notebook_1.ipynb notebook_2.ipynb