Skip to content

Commit bb2d341

Browse files
[Release] Update the cherry-pick process. (flutter#161771)
Updates to the Flutter cherry-pick process to align with our current practices. This update: 1. Defaults to automated cherry-picks. 2. Removes the need to create cherry-pick issues. 3. Requires cherry-pick authors to update CHANGELOG.md. --------- Co-authored-by: Christopher Fujino <[email protected]>
1 parent 3297454 commit bb2d341

File tree

6 files changed

+31
-110
lines changed

6 files changed

+31
-110
lines changed

.github/ISSUE_TEMPLATE/7_cherry_pick.yml

Lines changed: 0 additions & 90 deletions
This file was deleted.

docs/contributing/Design-Documents.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ If you want to write a design doc for people to review, we recommend using Googl
22
We have a template you can use, at [flutter.dev/go/template](https://flutter.dev/go/template). It describes the process for minting a `flutter.dev/go/foo` shortlink for your design doc.
33
We recommend you use that template so that people can immediately recognize that this is a Flutter design document and that it is shared publicly.
44

5-
After you have created your design doc, the next step is to create a tracking GitHub issue for it. File a new issue to track the design doc using [the design doc issue template](https://github.com/flutter/flutter/issues/new?template=8_design_doc.yml). Assign it to yourself. Add the label "design doc" to the issue.
5+
After you have created your design doc, the next step is to create a tracking GitHub issue for it. File a new issue to track the design doc using [the design doc issue template](https://github.com/flutter/flutter/issues/new?template=7_design_doc.yml). Assign it to yourself. Add the label "design doc" to the issue.
66

77
Don't forget to configure your document's Sharing settings so that everyone has comment access. The idea of sharing the document in this way is not necessarily to proactively obtain feedback from the entire community; it is to make it _possible_ for people to share the document with anyone in the community, whether they work for your employer or not, and whether you have personally shared the document with them yet or not.
88

docs/releases/Flutter-Cherrypick-Process.md

Lines changed: 30 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -6,27 +6,32 @@ With branching and branch testability being supported for Flutter & Dart release
66

77
**Note: This process applies to regressions from the previous release or serious bugs otherwise introduced by the current release. Feature work is not considered for cherry-picking and will need to wait for the next release.**
88

9-
## Automatically Creates a Cherry Pick Request
9+
## Automatically Create a Cherry-pick Request
1010

11-
Eligibility:
12-
1. If your cherry pick is expecting to have a merge conflict, please skip this section and follow instructions in the FAQ section below to manually open a cherry pick request instead. (e.g. PRs that contain changes to .ci.yaml files are very likely to hit a merge conflict)
13-
2. The framework PR to be cherry picked needs to have a base commit later than [01/24/2024](https://www.google.com/url?q=https://github.com/flutter/flutter/pull/142058&sa=D&source=docs&ust=1706904517596608&usg=AOvVaw3cFfw8vyiBtY3EzM_N-PEi), and the engine PR to be cherry picked has a base commit later than [02/06/2024](https://github.com/flutter/engine/pull/50265)
14-
3. The target branch is either [beta](https://github.com/flutter/flutter/blob/beta/bin/internal/release-candidate-branch.version) or [stable](https://github.com/flutter/flutter/blob/stable/bin/internal/release-candidate-branch.version). (not a new beta branch that isn't yet created)
15-
16-
For automatic cherry pick:
1711
1. Add the `cp: beta` or `cp: stable` label to the pull request on flutter/flutter master. (you can find [beta](https://github.com/flutter/flutter/blob/beta/bin/internal/release-candidate-branch.version) and [stable](https://github.com/flutter/flutter/blob/stable/bin/internal/release-candidate-branch.version) candidate branch info by following the respective links)
1812
2. Wait about 30 seconds.
1913
3. If automatic cherry pick succeeds (no merge conflict), a new pull requested will be created and you will receive an email. Edit the cherry-pick details in the PR description of the generated pull request, and a release engineer will follow up on the request.
20-
4. If automatic cherry pick fails, a comment will be left on the original PR. In this case you will need to follow instructions in the FAQ section below to manually create a cherry pick PR.
14+
4. If automatic cherry pick fails, a comment will be left on the original PR. In this case you will need to follow instructions in the manual cherry-pick section below to manually create a cherry pick PR.
15+
5. Update CHANGELOG.md following our [best practices](docs/releases/Hotfix-Documentation-Best-Practices.md).
2116

22-
For manual cherry pick:<br >
23-
refer to the FAQ section below
17+
If for some reason, an automated cherry-pick can not be applied, please follow the manual cherry-pick instructions.
2418

25-
## Frequently asked questions
19+
## Manually Create a Cherry-pick Request
20+
21+
If the automated cherry-pick process fails, you will have to create the cherry-pick request manually:
2622

27-
### How do I request a cherry-pick?
23+
1. Create a cherry-pick pull request to the intended branch. (you can find [beta](https://github.com/flutter/flutter/blob/beta/bin/internal/release-candidate-branch.version) and [stable](https://github.com/flutter/flutter/blob/stable/bin/internal/release-candidate-branch.version) candidate branch info by following the respective links).
24+
2. Edit the title of the cherry-pick request to start with either [beta] or [stable].
25+
3. Fill out the PR description with the following fields:
26+
- Impacted Users (Approximately who will hit this issue, ex. all Flutter devs, Windows developers, all end-customers, apps using X framework feature).
27+
- Impact Description (What is the impact? ex. visual jank on Samsung phones, app crash, cannot ship an iOS app. Does it impact development? ex. flutter doctor crashes when Android Studio is installed. Or shipping a production app? ex. the app crashes on launch).
28+
- Workaround (Is there a workaround for this issue?)
29+
- Risk (What is the risk level of this cherry-pick?)
30+
- Test Coverage (Are you confident that your fix is well-tested by automated tests?)
31+
- Validation Steps (What are the steps to validate that this fix works?)
32+
4. Ensure that your cherry-pick PR updates CHANGELOG.md.
2833

29-
To request a cherry-pick, utilize the [issue template](https://github.com/flutter/flutter/issues/new?template=7_cherry_pick.yml).
34+
## Frequently asked questions
3035

3136
### Who can request a cherry-pick?
3237

@@ -41,19 +46,25 @@ Anyone can request a cherry-pick.
4146

4247
The release engineering team will assign a cherry-pick reviewer who is an expert in the area of the code that your cherry-pick may affect.
4348

49+
### Why was my cherry-pick rejected
50+
51+
While we attempt to address every cherry-pick requests, there are various reasons a cherry-pick request may not be accepted to include, but not limited to:
52+
- Not filling out the pull request info appropriately.
53+
- Attempting to cherry-pick something other than a fix.
54+
- etc.
55+
4456
### Lifecycle of a cherry-pick
4557

4658
1. The cherry-pick requester opens a cherry-pick pull request to the [beta](https://github.com/flutter/flutter/blob/beta/bin/internal/release-candidate-branch.version) or [stable](https://github.com/flutter/flutter/blob/stable/bin/internal/release-candidate-branch.version) **candidate** branch (follow the respective link to find the branch name)
47-
2. A cherry-pick issue is filled out completely and created utilizing the [cherry-pick template](https://github.com/flutter/flutter/issues/new?template=7_cherry_pick.yml) in the [flutter/flutter](https://github.com/flutter/flutter) repository.
48-
3. The release engineering team is notified that a cherry-pick request is in queue and assigns an appropriate reviewer who is an expert in the area who will review the cherry-pick issue and associated cherry-pick pull request.
49-
4. The release engineering team applies the `merge-to-beta` or `merge-to-stable` label.
50-
5. The cherry-pick request then enters one of the following states.
59+
2. The release engineering team is notified that a cherry-pick request is in queue and assigns an appropriate reviewer who is an expert in the area who will review the cherry-pick issue and associated cherry-pick pull request.
60+
3. The release engineering team applies the `merge-to-beta` or `merge-to-stable` label.
61+
4. The cherry-pick request then enters one of the following states.
5162
1. Approved: The reviewer has approved the cherry-pick and cherry-pick pull request.
5263
The release engineering team will merge the cherry-pick pull request and apply the `cp: merged` label to the cherry-pick issue.
5364
2. Denied: The reviewer will comment on the cherry-pick issue why the cherry-pick is denied.
5465
The release engineering team will close the cherry-pick issue and associated cherry-pick pull request.
55-
6. The cherry-pick is picked up in the next release period.
56-
7. Once the cherry-pick has been added to a release, the release engineering team will close the cherry-pick issue.
66+
5. The cherry-pick is picked up in the next release period.
67+
6. Once the cherry-pick has been added to a release, the release engineering team will close the cherry-pick issue.
5768

5869
### This is my first cherry-pick, how do I do it?
5970

0 commit comments

Comments
 (0)