blob: 0bac643f41ef0234409f9cd506e0a82ecb63ce10 [file] [log] [blame]
= Gerrit Code Review - Contributing
== Contributor License Agreement
In order to contribute to Gerrit a link:dev-cla.html[Contributor
License Agreement] must be completed before contributions are accepted.
== Contribution Processes
The Gerrit project offers two contribution processes:
* link:#lightweight-contribution-process[Lightweight Contribution
* link:#design-driven-contribution-process[Design-Driven Contribution
The lightweight contribution process has little overhead and is best
suited for small contributions (documentation updates, bug fixes, small
features). Contributions are pushed as changes and
link:dev-roles.html#maintainer[maintainers] review them adhoc.
For large/complex features, it is required to follow the
link:#design-driven-contribution-process[design-driven contribution
process] and specify the feature in a link:dev-design-docs.html[design
doc] before starting with the implementation.
If link:dev-roles.html#contributor[contributors] choose the
lightweight contribution process and during the review it turns out
that the feature is too large or complex,
link:dev-roles.html#maintainer[maintainers] can require to follow the
design-driven contribution process instead.
If you are in doubt which process is right for you, consult the
mailing list.
These contribution processes apply to everyone who contributes code to
the Gerrit project, including link:dev-roles.html#maintainer[
maintainers]. When reading this document, keep in mind that maintainers
are also contributors when they contribute code.
If a new feature is large or complex, it is often difficult to find a
maintainer who can take the time that is needed for a thorough review,
and who can help with getting the changes submitted. To avoid that this
results in unpredictable long waiting times during code review,
contributors can ask for link:#mentorship[mentor support]. A mentor
helps with timely code reviews and technical guidance. Doing the
implementation is still the responsibility of the contributor.
=== Quick Comparison
| |Lightweight Contribution Process|Design-Driven Contribution Process
|Overhead|low (write good commit message, address review comments)|
high (write link:dev-design-docs.html[design doc] and get it approved)
|Technical Guidance|by reviewer|during the design review and by
|Review |adhoc (when reviewer is available)|by a dedicated mentor (if
a link:#mentorship[mentor] was assigned)
|Caveats |features may get vetoed after the implementation was already
done, maintainers may make the design-driven contribution process
required if a change gets too complex/large|design doc must stay open
for a minimum of 10 calendar days, a mentor may not be available
|Applicable to|documentation updates, bug fixes, small features|
large/complex features
=== Lightweight Contribution Process
The lightweight contribution process has little overhead and is best
suited for small contributions (documentation updates, bug fixes, small
features). For large/complex features the
link:#design-driven-contribution-process[design-driven contribution
process] is required.
As Gerrit is a code review tool, naturally contributions will
be reviewed before they will get submitted to the code base. To
start your contribution, please make a git commit and upload it
for review to the link:[] Gerrit server. To help speed up the
review of your change, review these link:dev-crafting-changes.html[
guidelines] before submitting your change. You can view the pending
Gerrit contributions and their statuses
Depending on the size of that list it might take a while for
your change to get reviewed. Naturally there are fewer
link:dev-roles.html#maintainer[maintainers], that can approve changes,
than link:dev-roles.html#contributor[contributors]; so anything that
you can do to ensure that your contribution will undergo fewer
revisions will speed up the contribution process. This includes
helping out reviewing other people's changes to relieve the load from
the maintainers. Even if you are not familiar with Gerrit's internals,
it would be of great help if you can download, try out, and comment on
new features. If it works as advertised, say so, and if you have the
privileges to do so, go ahead and give it a `+1 Verified`. If you
would find the feature useful, say so and give it a `+1 Code Review`.
And finally, the quicker you respond to the comments of your reviewers,
the quicker your change might get merged! Try to reply to every
comment after submitting your new patch, particularly if you decided
against making the suggested change. Reviewers don't want to seem like
nags and pester you if you haven't replied or made a fix, so it helps
them know if you missed it or decided against it.
=== Design-driven Contribution Process
The design-driven contribution process applies to large/complex
For large/complex features it is important to:
* agree on the functionality and scope before spending too much time
on the implementation
* ensure that they are in line with Gerrit's project scope and vision
* ensure that they are well aligned with other features
* think about possibilities how the feature could be evolved over time
This is why for large/complex features it is required to describe the
feature in a link:dev-design-docs.html[design doc] and get it approved
by the link:dev-processes.html#steering-committee[steering committee],
before starting the implementation.
The design-driven contribution process has the following steps:
* A link:dev-roles.html#contributor[contributor]
link:dev-design-docs.html#propose[proposes] a new feature by
uploading a change with a link:dev-design-docs.html[design doc].
* The design doc is link:dev-design-docs.html#review[reviewed] by
interested parties from the community. The design review is public
and everyone can comment and raise concerns.
* Design docs should stay open for a minimum of 10 calendar days so
that everyone has a fair chance to join the review.
* Within 14 calendar days the contributor should hear back from the
link:dev-processes.html#steering-committee[steering committee]
whether the proposed feature is in scope of the project and if it can
be accepted.
* To be submitted, the design doc needs to be approved by the
link:dev-processes.html#steering-committee[steering committee].
* After the design was approved, the implementation is done by pushing
changes for review, see link:#lightweight-contribution-process[
lightweight contribution process]. Changes that are associated with
a design should all share a common hashtag. The contributor is the
main driver of the implementation and responsible that it is done.
Others from the Gerrit community are usually much welcome to help
with the implementation.
In order to be accepted/submitted, it is not necessary that the design
doc fully specifies all the details, but the idea of the feature and
how it fits into Gerrit should be sufficiently clear (judged by the
steering committee). Contributors are expected to keep the design doc
updated and fill in gaps while they go forward with the implementation.
We expect that implementing the feature and updating the design doc
will be an iterative process.
While the design doc is still in review, contributors may already start
with the implementation (e.g. do some prototyping to demonstrate parts
of the proposed design), but those changes should not be submitted
while the design wasn't approved yet.
By approving a design, the steering committee commits to:
* Accepting the feature when it is implemented.
* Supporting the feature by assigning a link:dev-roles.html#mentor[
mentor] (if requested, see link:#mentorship[mentorship]).
If the implementation of a feature gets stuck and it's unclear whether
the feature gets fully done, it should be discussed with the steering
committee how to proceed. If the contributor cannot commit to finish
the implementation and no other contributor can take over, changes that
have already been submitted for the feature might get reverted so that
there is no unused or half-finished code in the code base.
For contributors, the design-driven contribution process has the
following advantages:
* By writing a design doc, the feature gets more attention. During the
design review, feedback from various sides can be collected, which
likely leads to improvements of the feature.
* Once a design was approved by the
link:dev-processes.html#steering-committee[steering committee], the
contributor can be almost certain that the feature will be accepted.
Hence, there is only a low risk to invest into implementing a feature
and see it being rejected later during the code review, as it can
happen with the lightweight contribution process.
* The contributor can link:#mentorship[get a dedicated mentor assigned]
who provides timely reviews and serves as a contact person for
technical questions and discussing details of the design.
== Mentorship
For features for which a link:dev-design-docs.html[design] has been
approved (see link:#design-driven-contribution-process[design-driven
contribution process]), contributors can gain the support of a mentor
if they are committed to implement the feature.
A link:dev-roles.html#mentor[mentor] helps with:
* doing timely reviews
* providing technical guidance during code reviews
* discussing details of the design
* ensuring that the quality standards are met (well documented,
sufficient test coverage, backwards compatible etc.)
A feature can have more than one mentor. To be able to deliver the
promised support, at least one of the mentors must be a
Mentors are assigned by the link:dev-processes.html#steering-committee[
steering committee]. To gain a mentor, ask for a mentor in the
Plan] section of the design doc or ask the steering committee after the
design has been approved.
Mentors may not be available immediately. In this case, the steering
committee should include the approved feature into the roadmap or
prioritize it in the backlog. This way, it is transparent for the
contributor when they can expect to be able to work on the feature with
mentor support.
Once the implementation phase starts, the contributor is expected to do
the implementation in a timely manner.
For every mentorship, the end must be clearly defined. The design doc
must specify:
* a maximum time frame for the mentorship, after which the mentorship
automatically ends, even if the feature is not done yet
* done criteria that define when the feature is done and the mentorship
If a feature is not finished in time, it should be discussed with the
steering committee how to proceed. If the contributor cannot commit to
finish the implementation in time and no other contributor can take
over, changes that have already been submitted for the feature might
get reverted so that there is no unused or half-finished code in the
code base.
Part of link:index.html[Gerrit Code Review]