Apache Ignite Developers - Legacy Mail Archive

Fwd: [CONF] Apache Ignite > How to Contribute

Classic

List

Threaded

1 message

Dmitriy Pavlov

Fwd: [CONF] Apache Ignite > How to Contribute

Hi Igniters,

Was it ever discussed we stop to require TC to be passed with PR?

I insist we keep this sentence here:

Any change that touches code or test, whether reviewed or not, must have full TeamCity CI test suite pass.

Sincerely,

Dmitriy Pavlov

---------- Forwarded message ---------
From: Vladimir Ozerov (Confluence) <[hidden email]>
Date: ср, 23 мая 2018 г. в 16:10
Subject: [CONF] Apache Ignite > How to Contribute
To: <[hidden email]>

There's 1 new edit on this page

Vladimir Ozerov edited this page

Here's what changed:

...

Ignite employs both Review-Then-Commit (RTC) process for most of the components and Commit-Then-Review (CTR) for few ones.
Any change that touches code or test, whether reviewed or not, must have full TeamCity CI test suite pass.
As a guideline, trivial changes, like updates to documentation or tests, etc. should not need a review prior to committing, and often can be made directly in the master branch.Please consult to Review Checklist to understand how tickets are reviewed and what rules to follow.

Info

title

Master Branch

Ignite "master" branch should always be release-ready. Please avoid any commits or merges to the "master" branch unless the whole TeamCity CI suite has passed.

...

Info

title

Optional: Tips to pass review quickly

Ask commiter to review changes directly.

Check affected files git history to find person most likely able to review changes.

In case it's hard to determine who's able to review by git history use maintainers list presented above.

Add "review request" comment to the Jira Issue starting with a commiter username.

for example: "[~avinogradov], Please review my changes."

Commiter will gain notification and review your changes and/or help to find another commiter to do this.

Reviewing a Ticket

Make sure that your patch satisfies Review Checklist rules.
Each comment should be started with [~username] to guarantee proper notification.
Commiter should add comment like "Changes accepted and ready to be merged." once review successfully finished.
If the commiter/contributor has proposals, it is recommended to add a comment in the ticket in addition to PR / CR comments referring proposals added. This helps other committers to identify that fix has something to improve.

...

•