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

Enable private field promotion #45722

Merged
merged 2 commits into from
Sep 14, 2023
Merged

Conversation

goderbauer
Copy link
Member

New feature in upcoming Dart 3.2. See dart-lang/language#2020. Feature is enabled by bumping the min SDK version to 3.2.

Part of flutter/flutter#134476.

@github-actions github-actions bot added platform-web Code specifically for the web engine platform-fuchsia e: impeller labels Sep 12, 2023
@goderbauer
Copy link
Member Author

Blocked on dart-lang/sdk#53505.

Copy link
Member

@chinmaygarde chinmaygarde left a comment

Choose a reason for hiding this comment

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

LGTM after the roll and presubs.

@goderbauer
Copy link
Member Author

Now waiting for dart-lang/sdk@b555e71 to roll in.

@goderbauer goderbauer added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 14, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Sep 14, 2023

auto label is removed for flutter/engine/45722, 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.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Sep 14, 2023
@goderbauer goderbauer added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 14, 2023
@auto-submit auto-submit bot merged commit 45bc430 into flutter:main Sep 14, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 14, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Sep 14, 2023
…134789)

flutter/engine@683bca5...45bc430

2023-09-14 [email protected] Enable private field promotion (flutter/engine#45722)
2023-09-14 [email protected] [Impeller] Add Entity::RenderingMode enum. (flutter/engine#45845)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Mairramer pushed a commit to Mairramer/flutter that referenced this pull request Oct 10, 2023
…lutter#134789)

flutter/engine@683bca5...45bc430

2023-09-14 [email protected] Enable private field promotion (flutter/engine#45722)
2023-09-14 [email protected] [Impeller] Add Entity::RenderingMode enum. (flutter/engine#45845)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
harryterkelsen pushed a commit that referenced this pull request Oct 23, 2023
New feature in upcoming Dart 3.2. See dart-lang/language#2020. Feature is enabled by bumping the min SDK version to 3.2.

Part of flutter/flutter#134476.
@goderbauer goderbauer deleted the privatFields branch March 8, 2024 22:23
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 e: impeller platform-fuchsia platform-web Code specifically for the web engine
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants