Skip to content

Conversation

@TahaTesser
Copy link
Member

Related to Introduce Material 3 year2023 flag to the updated widget themes

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Dec 3, 2024
@TahaTesser TahaTesser marked this pull request as ready for review December 3, 2024 14:52
@TahaTesser TahaTesser requested a review from QuncCccccc December 3, 2024 14:52
Copy link
Contributor

@QuncCccccc QuncCccccc left a comment

Choose a reason for hiding this comment

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

Thanks so much for the contribution! Looks good to me. Just left some nits in doc and comments in tests.

@TahaTesser TahaTesser requested a review from QuncCccccc December 5, 2024 12:15
Copy link
Contributor

@QuncCccccc QuncCccccc left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the help! Looks like we also have some conflict to fix:)

@TahaTesser TahaTesser force-pushed the year2023_slider_theme branch from a4d5020 to 340e12c Compare December 6, 2024 10:42
@TahaTesser TahaTesser enabled auto-merge December 6, 2024 14:45
@TahaTesser TahaTesser added this pull request to the merge queue Dec 6, 2024
Merged via the queue into flutter:master with commit ae736e2 Dec 6, 2024
34 of 78 checks passed
@TahaTesser TahaTesser deleted the year2023_slider_theme branch December 6, 2024 14:47
@TahaTesser
Copy link
Member Author

TahaTesser commented Dec 6, 2024

This is weird. This PR is merged before all the tests are done when in merge queue. Shouldn't it wait for all tests to pass before merging?

Update:

Post about this behavior on Discord infra channel.

@Piinks
Copy link
Contributor

Piinks commented Dec 6, 2024

The autosubmit label should still be used. The button is still the same as if it were squash and merge - you are manually overriding presubmit checks and putting it in the merge queue.

@TahaTesser
Copy link
Member Author

The autosubmit label should still be used. The button is still the same as if it were squash and merge - you are manually overriding presubmit checks and putting it in the merge queue.

Thanks for the info. I will keep using the autosubmit label.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
@reidbaker reidbaker mentioned this pull request Dec 13, 2024
11 tasks
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants