Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@bdero
Copy link
Member

@bdero bdero commented Apr 12, 2023

CP for #41101

@zanderso
Copy link
Member

zanderso commented Apr 12, 2023

I believe the correct branch is flutter-3.10-candidate.1. cc @CaseyHillers to confirm.

@CaseyHillers
Copy link
Contributor

Correct, the beta is based on 3.10-candidate.1. If you're aiming for a Google CP, please file a bug with more context.

@zanderso zanderso changed the base branch from flutter-3.10-candidate.3 to flutter-3.10-candidate.1 April 12, 2023 21:34
@zanderso
Copy link
Member

Ooooh. I tried just changing the target branch, but I think the branch for the pull request has to be rebased on candidate.1 as well.

@bdero bdero force-pushed the bdero/text-rounding-3.10-cp branch from 24e7e12 to 98b925a Compare April 12, 2023 22:51
@bdero
Copy link
Member Author

bdero commented Apr 12, 2023

Alrighty, fixed this up to point to candidate.1

@zanderso
Copy link
Member

@CaseyHillers do you know why the ci.yaml check is failing?

@bdero
Copy link
Member Author

bdero commented Apr 13, 2023

Looks like this is good to go

Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

release team will handle merging.

@CaseyHillers CaseyHillers merged commit bee4611 into flutter:flutter-3.10-candidate.1 Apr 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

embedder Related to the embedder API platform-fuchsia platform-ios platform-macos platform-web Code specifically for the web engine platform-windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants