An approval process for the change to the file path scope of global variables and configurations will be deployed for TiKV and PD repositories
These changes are scheduled to be deployed between 2024-05-30 ~ 2024-06-06.
Detail:
Some preparatory work needs to be completed first:
- The reviewing plug-in will be migrated to the native LGTM + approval plug-in, which is consistent with pingcap/tidb. This is the basis for implementation.
- Based on the code owner mechanism, setup approvers for changes to special files.
Some change will happen when it is deployed in the future days:
-
Add LGTM will shown as the
lgtm
label be added rather thanstatus/LGT1
orstatus/LGT2
. -
Approval of merging changes from the comment “/merge” to the comment “/approve”, and the github review approval of the approvers role will be regarded as approval by default.
-
The
lgtm
andapproved
label adding processes are in parallel, but they can only be merged when the final requirements are met.
Please take advantage one of the methods to handle a PR that is still in progress:
-
Turn the state of the pull request to draft state.
-
Add “WIP” to the beginning of the PR title.
-
Comment “/hold” in the pull request to hold it.
The final review habits will be consistent with “pingcap/tidb” and other repositories in “pingcap” GitHub organization.
Refs:
- Migrate to lgtm and approve plugins: LGTM plugin changes
- Code owner mechanism: TiDB code owners Proposal