This repository was archived by the owner on Nov 15, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
updating labels descriptions #13557
Merged
Merged
updating labels descriptions #13557
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
70ed72e
updating labels descriptions
the-right-joyce cf92c7f
delete milestones
the-right-joyce 9a32c86
Update docs/CONTRIBUTING.adoc
the-right-joyce 7335a26
link to label docs
the-right-joyce 7161cba
Merge remote-tracking branch 'origin/master' into the-right-joyce-upd…
0f91b4f
Update docs/CONTRIBUTING.adoc
the-right-joyce File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,26 +30,23 @@ A Pull Request (PR) needs to be reviewed and approved by project maintainers unl | |
|
|
||
| - `A-*` Pull request status. ONE REQUIRED. | ||
| - `B-*` Changelog and/or Runtime-upgrade post composition markers. ONE REQUIRED. (used by automation) | ||
| - `C-*` Release notes release-priority markers. EXACTLY ONE REQUIRED. (used by automation) | ||
| - `D-*` More general tags on the PR denoting various implications and requirements. | ||
| - `C-*` Release notes release-criticality markers. EXACTLY ONE REQUIRED. (used by automation) | ||
| - `D-*` Audit tags denoting auditing requirements on the PR. | ||
|
|
||
| *Process:* | ||
|
|
||
| . Please tag each PR with exactly one `A`, `B`, `C` and `D` label at the minimum. | ||
| . Once a PR is ready for review please add the https://github.com/paritytech/substrate/pulls?q=is%3Apr+is%3Aopen+label%3AA0-pleasereview[`A0-pleasereview`] label. Generally PRs should sit with this label for 48 hours in order to garner feedback. It may be merged before if all relevant parties had a look at it. | ||
| . If the first review is not an approval, swap `A0-pleasereview` to any label `[A3, A7]` to indicate that the PR has received some feedback, but needs further work. For example. https://github.com/paritytech/substrate/labels/A3-inprogress[`A3-inprogress`] is a general indicator that the PR is work in progress and https://github.com/paritytech/substrate/labels/A4-gotissues[`A4-gotissues`] means that it has significant problems that need fixing. Once the work is done, change the label back to `A0-pleasereview`. You might end up swapping a few times back and forth to climb up the A label group. Once a PR is https://github.com/paritytech/substrate/labels/A8-mergeoncegreen[`A8-mergeoncegreen`], it is ready to merge. | ||
| . PRs must be tagged with their release notes requirements via the `B1-B9` labels. | ||
| . PRs must be tagged with their release importance via the `C1-C9` labels. | ||
| . Once a PR is ready for review please add the https://github.com/paritytech/substrate/pulls?q=is%3Apr+is%3Aopen+label%3AA0-please_review+[`A0-please_review`] label. Generally PRs should sit with this label for 48 hours in order to garner feedback. It may be merged before if all relevant parties had a look at it. | ||
| . If the first review is not an approval, swap `A0-please_review` to any label `[A3, A5]` to indicate that the PR has received some feedback, but needs further work. For example. https://github.com/paritytech/substrate/labels/A3-in_progress[`A3-in_progress`] is a general indicator that the PR is work in progress. | ||
| . PRs must be tagged with their release notes requirements via the `B*` labels. If a PR should be mentioned in the release notes, use `B1` and add either: `T0-node`, `T1-runtime`or `T2-API` - this defines in which section of the release notes the PR will be shown. | ||
| . PRs must be tagged with their release importance via the `C1-C7` labels. | ||
| . PRs must be tagged with their audit requirements via the `D1-D9` labels. | ||
| . PRs that must be backported to a stable branch must be tagged with https://github.com/paritytech/substrate/labels/E1-runtimemigration[`E0-patchthis`]. | ||
| . PRs that introduce runtime migrations must be tagged with https://github.com/paritytech/substrate/labels/E1-runtimemigration[`E1-runtimemigration`]. See the https://github.com/paritytech/substrate/blob/master/utils/frame/try-runtime/cli/src/lib.rs#L18[Migration Best Practices here] for more info about how to test runtime migrations. | ||
| . PRs that introduce irreversible database migrations must be tagged with https://github.com/paritytech/substrate/labels/E2-databasemigration[`E2-databasemigration`]. | ||
| . PRs that add host functions must be tagged with with https://github.com/paritytech/substrate/labels/E4-newhostfunctions[`E4-newhostfunctions`]. | ||
| . PRs that break the external API must be tagged with https://github.com/paritytech/substrate/labels/E5-breaksapi[`E5-breaksapi`]. | ||
| . PRs that materially change the FRAME/runtime semantics must be tagged with https://github.com/paritytech/substrate/labels/E6-transactionversion[`E6-transactionversion`]. | ||
| . PRs that change the mechanism for block authoring in a backwards-incompatible way must be tagged with https://github.com/paritytech/substrate/labels/E7-breaksauthoring[`E7-breaksauthoring`]. | ||
| . PRs that "break everything" must be tagged with https://github.com/paritytech/substrate/labels/E8-breakseverything[`E8-breakseverything`]. | ||
| . PRs that block a new release must be tagged with https://github.com/paritytech/substrate/labels/E9-blocker%20%E2%9B%94%EF%B8%8F[`E9-blocker`]. | ||
| . PRs that introduce runtime migrations must be tagged with https://github.com/paritytech/substrate/labels/E0-runtime_migration[`E0-runtime_migration`]. See the https://github.com/paritytech/substrate/blob/master/utils/frame/try-runtime/cli/src/lib.rs#L18[Migration Best Practices here] for more info about how to test runtime migrations. | ||
| . PRs that introduce irreversible database migrations must be tagged with https://github.com/paritytech/substrate/labels/E1-database_migration[`E1-database_migration`]. | ||
| . PRs that add host functions must be tagged with with https://github.com/paritytech/substrate/labels/E3-host_functions[`E3-host_functions`]. | ||
| . PRs that break the external API must be tagged with https://github.com/paritytech/substrate/labels/F3-breaks_API[`F3-breaks_API`]. | ||
| . PRs that change the mechanism for block authoring in a backwards-incompatible way must be tagged with https://github.com/paritytech/substrate/labels/F1-breaks_authoring[`F1-breaks_authoring`]. | ||
| . PRs that "break everything" must be tagged with https://github.com/paritytech/substrate/labels/F0-breaks_everything[`F0-breaks_everything`]. | ||
| . PRs should be categorized into projects. | ||
| . No PR should be merged until all reviews' comments are addressed and CI is successful. | ||
|
|
||
|
|
@@ -91,21 +88,20 @@ To create a Polkadot companion PR: | |
| Note: The merge-bot currently doesn't work with forks on org accounts, only individual accounts. | ||
| (Hint: it's recommended to use `bot merge` to merge all substrate PRs, not just ones with a polkadot companion.) | ||
|
|
||
| If your PR is reviewed well, but a Polkadot PR is missing, signal it with https://github.com/paritytech/substrate/labels/A7-needspolkadotpr[`A7-needspolkadotpr`] to prevent it from getting automatically merged. | ||
| If your PR is reviewed well, but a Polkadot PR is missing, signal it with https://github.com/paritytech/substrate/labels/E6-needs_polkadot_pr[`E6-needs_polkadot_pr`] to prevent it from getting automatically merged. | ||
|
|
||
| As there might be multiple pending PRs that might conflict with one another, a) you should not merge the substrate PR until the Polkadot PR has also been reviewed and b) both should be merged pretty quickly after another to not block others. | ||
|
|
||
| == Helping out | ||
|
|
||
| We use https://github.com/paritytech/substrate/labels[labels] to manage PRs and issues and communicate state of a PR. Please familiarize yourself with them. Furthermore we are organizing issues in https://github.com/paritytech/substrate/milestones[milestones]. Best way to get started is to a pick a ticket from the current milestone tagged https://github.com/paritytech/substrate/issues?q=is%3Aissue+is%3Aopen+label%3AQ2-easy[`easy`] or https://github.com/paritytech/substrate/issues?q=is%3Aissue+is%3Aopen+label%3AQ3-medium[`medium`] and get going or https://github.com/paritytech/substrate/issues?q=is%3Aissue+is%3Aopen+label%3AX1-mentor[`mentor`] and get in contact with the mentor offering their support on that larger task. | ||
| We use https://github.com/paritytech/labels/blob/main/docs/doc_substrate.md[labels] to manage PRs and issues and communicate state of a PR. Please familiarize yourself with them. Furthermore we are organizing issues in https://github.com/paritytech/substrate/milestones[milestones]. Best way to get started is to a pick a ticket from the current milestone tagged https://github.com/paritytech/substrate/issues?q=is%3Aissue+is%3Aopen+label%3AZ1-easy[`easy`] or https://github.com/paritytech/substrate/issues?q=is%3Aissue+is%3Aopen+label%3AZ2-medium[`medium`] and get going or https://github.com/paritytech/substrate/issues?q=is%3Aissue+is%3Aopen+label%3AZ6-mentor[`mentor`] and get in contact with the mentor offering their support on that larger task. | ||
|
||
|
|
||
| == Issues | ||
| Please label issues with the following labels: | ||
|
|
||
| . `I-*` Issue severity and type. EXACTLY ONE REQUIRED. | ||
| . `P-*` Issue priority. AT MOST ONE ALLOWED. | ||
| . `Q-*` Issue difficulty. AT MOST ONE ALLOWED. | ||
| . `Z-*` More general tags on the issue, denoting context and resolution. | ||
| . `I-*` or `J-*` Issue severity and type. EXACTLY ONE REQUIRED. | ||
| . `U-*` Issue urgency, suggesting in what time manner does this issue need to be resolved . AT MOST ONE ALLOWED. | ||
the-right-joyce marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| . `Z-*` Issue difficulty. AT MOST ONE ALLOWED. | ||
|
|
||
| == Releases | ||
|
|
||
|
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.