Skip to content

Latest commit

 

History

History
99 lines (75 loc) · 3.15 KB

File metadata and controls

99 lines (75 loc) · 3.15 KB

Contributing

When contributing to this repository, please first discuss the change you wish to make via issue, email, or any other method with the owners of this repository before making a change.

Please note we have a code of conduct; please follow it in all your interactions with the project.

Table of Contents

  1. Pull Request Process
  2. Issue Report Process
  3. Commit Message Guidelines
  4. Code Quality

Pull Request Process

  1. Ensure any install or build dependencies are removed before the end of the layer when doing a build.
  2. Ensure that tests and linting pass.
  3. Increase the version numbers in any examples files and the README.md that this Pull Request would represent. The versioning scheme we use is SemVer.
  4. You may merge the Pull Request once you have the sign-off of two other developers, or if you do not have permission to do that, you may request the second reviewer to merge it for you.

Issue Report Process

  1. Go to the project's issues.
  2. Select the template that better fits your issue.
  3. Read the instructions carefully and write within the template guidelines.
  4. Submit it and wait for support.

Commit Message Guidelines

When committing, commit messages are prefixed with one of the following depending on the type of change made, adhering to the Conventional Commits standard.

  • feat: when a new feature is introduced with the changes.
  • fix: when a bug fix has occurred.
  • chore: for changes that do not relate to a fix or feature and do not modify source or tests. (like updating dependencies)
  • refactor: for refactoring code that neither fixes a bug nor adds a feature.
  • docs: when changes are made to documentation.
  • style: when changes that do not affect the code, but modify formatting.
  • test: when changes to tests are made.
  • perf: for changes that improve performance.
  • ci: for changes that affect CI.
  • build: for changes that affect the build system or external dependencies.
  • revert: when reverting changes.

Commit messages are also to begin with an uppercase character. Below list some example commit messages.

git commit -m "docs: Added README.md"
git commit -m "revert: Removed README.md"
git commit -m "docs: Moved README.md"

Code Quality

Testing

We use Pytest to test our code. Please ensure that tests are updated and pass before merging a Pull Request.

# To test your code, run:
npm test

# Or
pytest -sv .

Linting

We use Ruff, Eslint and Prettier to ensure that code is consistent and follows our code style. Please ensure that your code passes linting before merging a Pull Request.

# To lint your code, run:
npm run lint

# To fix any linting errors, run:
npm run lint:fix