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

Conversation

@chinmaygarde
Copy link
Contributor

Reverts #8616

This breaks the bots because the flow namespace that you removed was used. I think it was just the patches landing in the wrong order.

@chinmaygarde
Copy link
Contributor Author

TBR. Landing on read as it fixes the bots.

Copy link
Contributor

@mklim mklim left a comment

Choose a reason for hiding this comment

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

LGTM

@chinmaygarde chinmaygarde merged commit 91b7107 into master Apr 18, 2019
@chinmaygarde chinmaygarde deleted the revert-8616-total_elevation_test branch April 18, 2019 00:48
@liyuqian
Copy link
Contributor

Oops, thanks for catching this and reverting. Will fix the namespace later.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 18, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 18, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 18, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 18, 2019
flutter/engine@876c4c3...8b5a50c

git log 876c4c3..8b5a50c --no-merges --oneline
8b5a50c Test saving compilation traces. (flutter/engine#8618)
7d3caf8 Avoid leaking the VM in runtime_unittests and update failing tests. (flutter/engine#8626)
91b7107 Revert "Add a unit test for PhysicalShapeLayer (#8616)" (flutter/engine#8627)
8b667b0 Add a unit test for PhysicalShapeLayer (flutter/engine#8616)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants