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

Conversation

@matanlurey
Copy link
Contributor

@matanlurey matanlurey commented Nov 16, 2023

Includes a test (header_filter_regex_test.dart) that hopefully keeps us on rails going forward.

@matanlurey matanlurey added the test: all See https://github.com/flutter/engine/blob/main/docs/ci/Engine-pre-submits-and-post-submits.md label Nov 16, 2023
@matanlurey matanlurey marked this pull request as ready for review November 27, 2023 17:21
@matanlurey matanlurey requested a review from zanderso November 27, 2023 17:22
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Nov 27, 2023

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

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Nov 27, 2023

auto label is removed for flutter/engine/48145, due to - The status or check suite Linux Framework Smoke Tests has failed. Please fix the issues identified (or deflake) before re-applying this label.

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Nov 27, 2023

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

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Nov 27, 2023

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

  • The status or check suite Mac mac_clang_tidy_presubmit has failed. Please fix the issues identified (or deflake) before re-applying this label.

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Nov 27, 2023

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

  • The status or check suite Mac mac_clang_tidy_presubmit has failed. Please fix the issues identified (or deflake) before re-applying this label.

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@matanlurey matanlurey merged commit 5ffe1f4 into flutter:main Nov 27, 2023
@matanlurey matanlurey deleted the engine-clangTidy-vFinal branch November 27, 2023 23:30
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 28, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 28, 2023
@bdero bdero mentioned this pull request Nov 29, 2023
8 tasks
caseycrogers pushed a commit to caseycrogers/flutter that referenced this pull request Dec 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

affects: desktop autosubmit Merge PR when tree becomes green via auto submit App e: impeller platform-ios platform-macos test: all See https://github.com/flutter/engine/blob/main/docs/ci/Engine-pre-submits-and-post-submits.md

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants