2.6 KiB
Contributing
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.
Proposing changes
The main branch is protected and cannot be committed to directly. It will always contain the latest version of the project that is considered to be stable. Development happens on the dev branch. In order to propose a change to the project, we have the following process:
- Fork the repository if you do not have write access.
- Create a branch from
dev, using the following naming convention:feat/feature-namewhen introducing a new feature.fix/fix-namewhen fixing an existing issue.chore/chore-decfor miscellaneous changes.
- Stick to the project coding style. (See below.)
- Add your commits to the branch following Conventional Commits (See below.)
- Open a Pull Request targeting the
devbranch and tag the right code owners for review. (See below.) - Merge the PR into
devif you have write access or request a merge from a code owner after reviews and checks have passed.
Note that any new code must be covered by unit tests. Only PRs with checks successfully are eligible to be merged. The code owners reserve the right to accept or reject PRs.
Any contributions that are accepted become part of the code base under the same license that covers the project.
Opening issues
You can open an issue via GitHub's issue tracker. You may tag one or more code owners for visibility if you think it fits the issue.
Coding style
- Tabs instead of spaces.
- Closing brackets generally go on a new line.
Conventional Commits
We follow the Conventional Commits standard for commit messages. The pattern for such a commit message is as follows:
type(scope?): description
Where type is one of:
feat: commits that add a feature.fix: commits that fix an issue.refactor: commits that (re)write code but do not change behaviour.chore: miscellaneous commits or housekeeping.- Or one of the other types described by the standard.
The scope portion is optional and valid options depend on the project.
Example Conventional Commit message:
fix(yield-token): unwrap memo and print if not none
Code owners
All repositories should contain a CODEOWNERS file that automatically requests the right code owner(s) for review when a new PR is submitted. If it does not happen, or if you think someone else should be included, you can request a review manually.