Skip to content

Commit 3f7daa0

Browse files
authored
docs: clarify pull request review process (#110)
1 parent cb1d498 commit 3f7daa0

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

docs/PULL_REQUESTS.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Pull requests
22

3-
To add code to the different repositories the code is build in a feature branch and merged back to the branch ``develop``
3+
To add code to the different repositories the code is build in a feature branch and merged back to the ``develop`` or `` main`` branch.
44
using a pull request. No direct committing is allowed to the branches ``develop`` and ``main``.
55
When a release is created the branch ``develop`` is merged to the branch ``main`` using a pull request.
66
In some repositories the branch ``develop`` is skipped and not there. Feature branches are merged directly to the branch
@@ -27,4 +27,9 @@ The following labels cause the pull request to be ignored in the release notes:
2727
- ``duplicate``
2828
- ``invalid``
2929

30-
**Remark**: The label ``tooling`` is not a standard label, so that may need to be created by the first developer using it.
30+
**Remark**: The label ``tooling`` is not a standard label, so that may need to be created by the first developer using it.
31+
32+
## Code review
33+
34+
If you are a long time and trusted contributor, your can review your pull-requests with your collegues.
35+
Otherwise request the code owner of the repository to review your pull request. If you haven't heard back in 3 days, report your pull request in the [CoMPAS slack channel](https://lfenergy.slack.com/)

0 commit comments

Comments
 (0)