[
https://issues.apache.org/jira/browse/IGNITE-19883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17746866#comment-17746866
]
Sergey Chugunov commented on IGNITE-19883:
------------------------------------------
[~slava.koptilin], I expanded Formal Criteria section with more detailed items,
could you please take a look once again?
> Add a review checklist to a PR in Apache Ignite 3 repository
> ------------------------------------------------------------
>
> Key: IGNITE-19883
> URL: https://issues.apache.org/jira/browse/IGNITE-19883
> Project: Ignite
> Issue Type: Task
> Reporter: Sergey Chugunov
> Assignee: Sergey Chugunov
> Priority: Major
> Labels: ignite-3
> Fix For: 3.0.0-beta2
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> We need to add a checklist of bullets to verify PR against right to the PRs
> pages in GitHub, like a list already implemented for PRs in AI2 repo:
> {noformat}
> 1. There is a single JIRA ticket related to the pull request.
> 2. The web-link to the pull request is attached to the JIRA ticket.
> 3. The JIRA ticket has the Patch Available state.
> 4. The pull request body describes changes that have been made. The
> description explains WHAT and WHY was made instead of HOW.
> 5. The pull request title is treated as the final commit message. The
> following pattern must be used: IGNITE-XXXX Change summary where XXXX -
> number of JIRA issue.
> 6. A reviewer has been mentioned through the JIRA comments (see the
> Maintainers list)
> 7. The pull request has been checked by the Teamcity Bot and the green visa
> attached to the JIRA ticket (see TC.Bot: Check PR)
> {noformat}
> Though in AI2 checklist if focused mostly on formal stuff, and we want to
> implement the list more dedicated to code quality and easy-to-review reqs.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
