Revision 8780ab23e2e8a70ca2d4730f6ad1d8a4222f02f7 authored by Sylvain Ribstein on 10 July 2023, 09:07:21 UTC, committed by Marge Bot on 18 July 2023, 21:46:02 UTC
1 parent 9530c2f
merge_team.rst
Octez Merge Team
================
The Octez merge team is a `group of developers
<https://gitlab.com/groups/tezos/-/group_members>`_
from different teams of the
Tezos ecosystem.
They are automatically selected as approvers when you create a merge
request.
Octez Merge Team Scope
----------------------
The role of the Octez Merge Team is limited to ensuring the quality
of what gets merged into Octez, i.e. all of `tezos/tezos
<https://gitlab.com/tezos/tezos>`_ except :src:`src/proto_alpha/lib_protocol/`.
By quality, we mean ensuring that the code is as good as it can be, be it
at the implementation level or the architecture level.
The Octez merge team does not decide what contributions are included in
the next protocol (``src/proto_alpha/lib_protocol/``). Companies that contribute to
`tezos/tezos <https://gitlab.com/tezos/tezos>`_
reach a consensus to decide what gets in the Alpha protocol,
i.e. in the proposal of the next upgrade. In particular, other companies
can fork this repository and do their own protocol proposals.
.. _merge_coordinator:
Merge Coordinator
-----------------
The *merge coordinator* is a member of the Octez merge team whose role
is to make sure the merge request (MR) process is going smoothly. Duties
of the merge coordinator include:
* Make sure merge requests follow the :ref:`MR workflow <mr_workflow>`:
the :ref:`assignee <assignee_field>` is correct, the :ref:`reviewers <reviewers_field>`
are correct; and the :ref:`draft <draft_mode>` status is correct.
* Supervise ready MRs to ensure that the right actions are being taken;
advise how to make them move forward.
* Assist with removing blockers, organize the optimal sequencing of MRs.
* Assist with notifying blocking persons that they are a bottleneck:
find MRs blocked on review and ask updates to the blocking reviewer,
find another reviewer if relevant.
* Actually perform MR reviews/approvals where appropriate, upgrade
approvals from outside the Octez merge team when relevant.
* Coordinate with and alert developers working on CI in the case of problems
that affect the flow of MRs.
* Help MR authors understand their responsibilities.
* Coordinate with leaders of the various companies of the Tezos ecosystem to
make sure the process is understood and shared.
* Improve and augment the documentation about the MR process.
To make it easy to find out who the current merge coordinator is,
:gl:`meta-issue 1062 <tezos/tezos#1062>` is assigned to the merge coordinator.
Don't hesitate to contact the merge coordinator if you think you can
benefit of their help to move your merge request forward.
Getting into the Octez Merge Team
---------------------------------
The Octez merge team is always looking for software engineers with at least the following qualities:
- You are an active developer on the `tezos/tezos
<https://gitlab.com/tezos/tezos>`_ repository.
- You have done numerous reviews and merge requests in the past, i.e. you have
shown your expertise and interest in contributing both with code and reviews.
- You are paying attention to detail and are on the lookout for possible
errors, security issues, and/or enhancements in your reviews. It's fine not knowing
the entire codebase (no one does), but you should nevertheless
actively try to make the code of others better when you do reviews.
- You are well capable in software design and can identify subpar design choices.
- One way to show attention to detail and to design is to show these
qualities in your own merge requests. You should also be willing to amend your merge
requests by taking into account the feedback of reviewers, be it
for small changes or architectural changes.
- You know when your expertise is insufficient and you are keen to
ask for support.
- You are relatively skilled in one of the technologies used in the ``tezos/tezos``
repository, i.e. ``OCaml``, ``python``, ``CI``, building, packaging, etc.
To apply for being included in the Octez merge team, contact the :ref:`merge coordinator <merge_coordinator>` or
an existing member on the `tezos-dev <https://tezos-dev.slack.com/>`_ Slack. Your application
will be discussed during the next weekly meeting of the Octez merge team.
Helping the Octez Merge Team
----------------------------
Even if you are not in the Octez merge team, you can help it! If you review a merge
request and you think it is ready to be merged, you should click the *Approve* button
even if your approval "doesn't count".
If the merge request is not critical,
or if you are knowledgeable on the merge request topic; your approval
will make it easier for an Octez merge team member to approve too.
This is also an excellent way to show your skills and willingness to move development
forward, contributing to potentially making you an Octez merge team member in the future
(see *Getting into the Octez Merge Team* above).
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...