Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[$RELEASE_BRANCH-cp] Cherry pick #139405

Closed

Conversation

XilaiZhang
Copy link
Contributor

@XilaiZhang XilaiZhang commented Dec 2, 2023

name: Request a cherry-pick
description: As a contributor, you would like to request that a feature be cherry-picked into a release.
title: '[CP] <title>'
labels: ['cp: review']
body:

  • type: input
    id: issue_link
    attributes:
    label: Issue Link
    description: What is the link to the issue this cherry-pick is addressing?
    validations:
    required: true
  • type: input
    id: commit_hash
    attributes:
    label: Commit Hash
    description: What is the shortened commit hash that has been merged to master/main?
    validations:
    required: true
  • type: dropdown
    id: Target
    attributes:
    label: Target
    description: Should this be cherry-picked to beta or stable?
    options:
    - stable
    - beta
    validations:
    required: true
  • type: input
    id: pr_link
    attributes:
    label: PR Link
    description: >-
    Link to an open PR that cherrypick's this into the target release branch.
    The current branches can be found under release-caniddate-branch.version for beta and stable
    validations:
    required: true
  • type: textarea
    id: changelog_description
    attributes:
    label: Changelog Description
    description: >-
    Explain this CP in one line that is accessible to most Flutter developers
    See https://github.com/flutter/flutter/wiki/Hotfix-Documentation-Best-Practices for examples
    validations:
    required: true
  • type: input
    id: impacted_users
    attributes:
    label: Impacted Users
    description: Approximately who will hit this issue (ex. all Flutter devs, Windows developers, all end-customers, apps using X framework feature)?
    validations:
    required: true
  • type: textarea
    id: impact_description
    attributes:
    label: Impact Description
    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 the shipping production app (the app crashes on launch)
    validations:
    required: true
  • type: textarea
    id: workaround
    attributes:
    label: Workaround
    description: Is there a workaround for this issue?
    validations:
    required: true
  • type: dropdown
    id: risk
    attributes:
    label: Risk
    description: What is the risk level of this cherry-pick?
    options:
    - low
    - medium
    - high
    validations:
    required: true
  • type: dropdown
    id: test_coverage
    attributes:
    label: Test Coverage
    description: Are you confident that your fix is well-tested by automated tests?
    options:
    - "yes"
    - "no"
    validations:
    required: true
  • type: textarea
    id: fix_steps
    attributes:
    label: Validation Steps
    description: What are the steps to validate that this fix works?

mdebbar and others added 2 commits December 2, 2023 01:17
The tree is red due a non-hermetic customer test from `macos_ui`. The
test was disabled in flutter/tests#317 and an
issue was filed at macosui/macos_ui#499.

We need this empty commit in order to re-run CI with the latest
https://github.com/flutter/tests.
@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Dec 2, 2023
@XilaiZhang
Copy link
Contributor Author

The template looks very wrong, i will look more into this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants