Skip to content

Latest commit

 

History

History
101 lines (60 loc) · 7.77 KB

CONTRIBUTING.md

File metadata and controls

101 lines (60 loc) · 7.77 KB

Contributing

👍🎉 First off, yes, you can contribute and thanks already for taking the time if you do! 🎉👍

How we organize code

To keep track of different states of our code (current release, bugfixes, features) we use branches:

Branch Used for PRs allowed?
main Latest released version
develop-patch Working branch for next patch release, e.g. 4.0.x
develop-minor Working branch for next minor release, e.g. 4.x.0
v5/develop Working branch for next major release, e.g. 5.0.0
fix/* Temporary branches for single bugfix -
feature/* Temporary branches for single feature -
release/* Pre-releases in testing before they are merged into main when released only during release testing

We will review all pull requests (PRs) to develop-patch, develop-minor and v5/develop and merge them if accepted, once an appropriate version is upcoming. Please understand that this might not be the immediate next release and might take some time.

How you can contribute

Report a bug

When you find a bug, the first step to fixing it is to help us understand and reproduce the bug as best as possible. When you create a bug report, please include as many details as possible. Fill out the template because the requested information helps us resolve issues so much faster.

Bug fixes

For bug fixes, please create a new branch following the name scheme: fix/issue_number-bug-x, e.g. fix/234-this-nasty-bug. Limit bug fix PRs to a single bug. Do not mix multiple bug fixes in a single PR. This will make it easier for us to review the fix and merge it.

  • Always send bug fix PRs against the develop-patch branch––not main.
  • Add a helpful description of what the PR does if it is not 100% self-explanatory.
  • Every bug fix should include a unit test to avoid future regressions. Let us know if you need help with that.
  • Make sure your code style matches ours and includes comments/in-code documentation.
  • Make sure your branch is up to date with the latest state on the develop-patch branch. Rebase changes before you send the PR.
  • Please don't commit updated dist files in the panel/dist folder to avoid merge conflicts. We only build the dist files on release. Your branch should only contain changes to the source files.

Features

For features create a new branch following the name scheme: feature/issue_number-feature-x, e.g. feature/123-awesome-function. Our feedback platform can be a good source of highly requested features. Maybe your feature idea already exists and you can get valuable feedback from other Kirby users. Focus on a single feature per PR. Don't mix features!

  • Always send feature PRs against the develop-minor branch––not main.
  • Add a helpful description of what the PR does.
  • New features should include unit tests. Let us know if you need help with that.
  • Make your code style matches ours and includes comments/in-code documentation.
  • Make sure your branch is up to date with the latest state on the develop-minor branch. Rebase changes before you send the PR.
  • Please don't commit updated dist files in the panel/dist folder to avoid merge conflicts. We only build the dist files on release. Your branch should only contain changes to the source files.

We try to bundle features in our major releases, e.g. 5.0. That is why we might only review and, if accepted, merge your PR once an appropriate release is upcoming. Please understand that we cannot merge all feature ideas or that it might take a while. Check out the roadmap to see upcoming releases.

Translations

We are really happy about any help with translations. Please do not directly translate JSON files, though. We use a service called Transifex to handle all translations. Create an account there and send us a request to join our translator group. Additionally, also send an email to support@getkirby.com. Unfortunately, we don't get notified properly about new translator requests.

How we write code

Style

Backend (PHP)

We use PHP CS Fixer to ensure a consistent style for our PHP code. It is mainly based on PSR-12. Install PHP CS Fixer globally via Composer and then run composer fix in the kirby folder to check for inconsistencies and fix them. Our automated PR checks will fail if there are code style issues with your code.

Frontend/Panel (JavaScript, Vue)

We use Prettier to ensure a consistent style for our JavaScript and Vue code. After running npm install in the kirby/panel folder, you can run npm run format to check for inconsistencies and fix them. We also use ESLint which you can use by running npm run lint and/or npm run lint:fix.

Documentation

In-code documentation and comments help us understand each other's code - or our own code after some months. Especially when matters get more complicated, we try to add a lot of comments to explain what the code does or why we implemented it like this. Even better than good comments is good code that is easy to understand.

Backend (PHP)

We use PHP DocBlocks for classes and methods.

Frontend/Panel (JavaScript, Vue)

We use JSDoc for documenting JavaScript code, especially for Vue components.

Public documentation

We also document Kirby on the Kirby website at https://getkirby.com. However we recommend to wait with writing public documentation until the feature PR is merged. If you don't know where the documentation for a feature best belongs, don't worry. We can take care of writing the docs.

Tests

Unit and integration tests help us prevent regressions when we make changes to the code. Every bug fix should also add a unit test for the fixed bug to make sure we won't re-introduce the same problem later down the road. Every new feature should be accompanied by unit tests to protect it from breaking through future changes.

Backend (PHP)

We use PHPUnit for unit test for our PHP code. You can find all existing tests in the kirby/tests subfolders. Take a look to see how we usually structure our tests.

Frontend/Panel (JavaScript, Vue)

The Panel doesn't have extensive test coverage yet. That's an area we are still trying to improve.

We use vitest for unit tests for JavaScript and Vue components - .test.js files next to the actual JavaScript/Vue file.

And last…

Let us know in the forum if you have questions.

And once more: thank you! 👍🎉