Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Revert "Manually roll Flutter Engine from cbb9d0b06 to cef721963" #112902

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

zanderso
Copy link
Member

@zanderso zanderso commented Oct 5, 2022

Reverts #112895

This engine roll picked up a Dart SDK roll. The SDK roll had a pub roll in it. Some pub invocations on CI are now failing like:

https://ci.chromium.org/ui/p/flutter/builders/prod/Linux%20technical_debt__cost/8557/overview

@flutter-dashboard flutter-dashboard bot added the engine flutter/engine repository. See also e: labels. label Oct 5, 2022
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@zanderso zanderso added revert Autorevert PR (with "Reason for revert:" comment) warning: land on red to fix tree breakage Override tree-status signal (land even with closed tree), combine with the autosubmit label. labels Oct 5, 2022
@auto-submit auto-submit bot merged commit 4e70baa into master Oct 5, 2022
@auto-submit auto-submit bot deleted the revert-112895-engine-roll-revert branch October 5, 2022 03:03
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 5, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 5, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 5, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 5, 2022
Copy link
Contributor

@Hixie Hixie left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engine flutter/engine repository. See also e: labels. revert Autorevert PR (with "Reason for revert:" comment) warning: land on red to fix tree breakage Override tree-status signal (land even with closed tree), combine with the autosubmit label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants