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

Macos re-enable TestSetMenu #37058

Merged
merged 1 commit into from
Oct 27, 2022
Merged

Conversation

cyanglaz
Copy link
Contributor

Re-enables the TestSetMenu test.
The workaround in flutter/flutter#104748 (comment) is applied in this PR.

Fixes flutter/flutter#104748

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on
    writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

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

@cyanglaz cyanglaz requested a review from jmagman October 26, 2022 22:01
@cyanglaz cyanglaz marked this pull request as ready for review October 26, 2022 22:04
Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

LGTM stamp from a Japanese personal seal

@cyanglaz cyanglaz added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 26, 2022
@auto-submit
Copy link
Contributor

auto-submit bot commented Oct 26, 2022

auto label is removed for flutter/engine, pr: 37058, due to - The status or check suite Mac Host clang-tidy has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 26, 2022
@cyanglaz cyanglaz added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 27, 2022
@auto-submit auto-submit bot merged commit c33eae1 into flutter:main Oct 27, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 27, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 27, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 27, 2022
christopherfujino pushed a commit to flutter/flutter that referenced this pull request Oct 27, 2022
…114178)

* c33eae133 Macos re-enable TestSetMenu (flutter/engine#37058)

* abc3aabad PlatformView partial blur (flutter/engine#36015)

* 8fcf413f8 Revert "PlatformView partial blur" (flutter/engine#37085)

* 8bb0441e5 Revert "Fixes Android text field to use hint text for accessibility (#36846)" (flutter/engine#37083)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-macos
Projects
None yet
3 participants