Skip to content

Latest commit

 

History

History
248 lines (183 loc) · 7.44 KB

CONTRIBUTING.md

File metadata and controls

248 lines (183 loc) · 7.44 KB

Contributing to incendium

We follow a simple model to make contributing as straightforward as possible. These guidelines allow us to streamline the development process and achieve greater transparency.

Code of Conduct

Help us keep incendium open and inclusive. Please read and follow our Code of Conduct.

Got a question?

Please join us on or Discussions.

Found a bug?

If you find a bug or if something is missing in the source code, you can help us by submitting an issue, or even better, you can submit a Pull Request with a fix.

Getting ready to contribute

For incendium we rely on Python 2.7.18 for development, and Python 3.9+ to run tests and style checks with pre-commit and tox.

Setting up your local environment

  1. Install Python 2.7.18 and the 3.9+

  2. Install the required packages for development you may run the following command:

    python2 -m pip install --requirement requirements.txt

  3. Install Python 3 tools

    1. pre-commit

      python3 -m pip install pre-commit

      1. Install the git hook scripts

        pre-commit install

    2. tox

      python3 -m pip install tox

Pull Requests

We use the GitHub flow as main versioning workflow.

  1. Fork the repository
  2. Create a new branch from main for each feature, fix or improvement, using the following naming convention:
    1. feat/scope/feature-name when introducing a new feature
    2. fix/scope/fix-name when fixing an existing issue
    3. docs/change for documentation changes
    4. chore/chore-desc for miscellaneous changes
  3. Make sure to install pre-commit and run our pre-commit hooks and apply our Coding Style.
  4. Add your commits to the branch following our Commit Message Format
  5. Send a pull request from each feature branch to the main branch

It is very important to separate new features or improvements into separate feature branches, and to send a pull request for each branch.

This allows us to review and pull in new features or improvements individually.

Coding style

We use the Black code style, but also rely on other tools.

Commit Message Format

We have very precise rules over how our Git commit messages must be formatted. This format leads to easier to read commit history.

Each commit message consists of a header, a body, and a footer, trying to adhere to the "50/72 rule".

<header>
<BLANK LINE>
<body>
<BLANK LINE>
<footer>

The header is mandatory and must conform to the Conventional Commits specification. Additionally, we recommend trying to keep the header to a maximum of 50 characters.

The body is mandatory for all commits except for those of type "docs". When the body is present it should be wrapped at 72 characters.

The footer is optional. The Commit Message Footer format describes what the footer is used for and the structure it must have.

Commit message header

<type>(<scope>): <summary>
  │       │             │
  │       │             └─⫸ Summary in present tense. Not capitalized. No period
  |       |                  at the end.
  │       │
  │       └─⫸ Commit Scope: constants|dataset|date|db|exceptions|gui
  |                          helper|l10n|net|tag|time|types|user|vision
  │
  └─⫸ Commit Type: build|ci|docs|feat|fix|perf|refactor|test

The <type> and <summary> fields are mandatory, the (<scope>) field is optional.

Type

Must be one of the following:

  • build: Changes that affect the build system or external dependencies (example scope: deps, pip)
  • chore: Other changes that don't modify src or test files (example scopes: release)
  • ci: Changes to our CI configuration files and scripts (example scope: pip)
  • docs: Documentation only changes
  • feat: A new feature
  • fix: A bug fix
  • perf: A code change that improves performance
  • refactor: A code change that neither fixes a bug nor adds a feature
  • revert: Reverts a previous commit
  • style: Changes that do not affect the meaning of the code (white-space, formatting, etc.)
  • test: Adding missing tests or correcting existing tests

Scope

The scope should be the name of the Python package affected.

The following is the list of supported scopes:

  • constants
  • dataset
  • date
  • db
  • exceptions
  • l10n
  • net
  • tag
  • time
  • types (found under incendium.helper.types)
  • user
  • vision

There are currently a few exceptions to the "use package name" rule:

  • deps: for updating dependencies for our CI scripts. This is mainly used by dependabot and pre-commit.ci
  • pip: for changes to setup.[cfg|py], or the package publishing workflow
  • release: used for creating a new release
  • none/empty string: useful for changes that are done across all packages (e.g. style: use black style) and for docs changes that are not related to a specific package (e.g. docs: fix typo in README).

Summary

Use the summary field to provide a succinct description of the change:

  • use the imperative, present tense: "change" not "changed" nor "changes"
  • don't capitalize the first letter
  • no dot (.) at the end

Example Conventional Commit message:

refactor(dataset): use Dataset instance functions

Commit Message Footer

The footer can contain information about breaking changes and deprecations and is also the place to reference GitHub issues, and other PRs that this commit closes or is related to. For example:

BREAKING CHANGE: <breaking change summary>
<BLANK LINE>
<breaking change description + migration instructions>
<BLANK LINE>
<BLANK LINE>
Fixes #<issue number>

or

DEPRECATED: <what is deprecated>
<BLANK LINE>
<deprecation description + recommended update path>
<BLANK LINE>
<BLANK LINE>
Closes #<pr number>

Breaking Change section should start with the phrase "BREAKING CHANGE: " followed by a summary of the breaking change, a blank line, and a detailed description of the breaking change that also includes migration instructions.

Similarly, a Deprecation section should start with "DEPRECATED: " followed by a short description of what is deprecated, a blank line, and a detailed description of the deprecation that also mentions the recommended update path.

Revert commits

If the commit reverts a previous commit, it should begin with revert:, followed by the header of the reverted commit.

The content of the commit message body should contain:

  • information about the SHA of the commit being reverted in the following format: This reverts commit <SHA>,
  • a clear description of the reason for reverting the commit message.

Example:

revert(ci): remove condition from job

Refs: 2a50de4