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

Roll dart e182eac158cf..9bc7d4604277#22332

Merged
fluttergithubbot merged 2 commits intoflutter:masterfrom
iskakaushik:fix-dart-roll
Nov 5, 2020
Merged

Roll dart e182eac158cf..9bc7d4604277#22332
fluttergithubbot merged 2 commits intoflutter:masterfrom
iskakaushik:fix-dart-roll

Conversation

@iskakaushik
Copy link
Contributor

@iskakaushik iskakaushik commented Nov 5, 2020

dart.googlesource.com/sdk.git/+log/e182eac158cf..9bc7d46

2020-11-05 dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com Version 2.12.0-24.0.dev
2020-11-05 dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com Version 2.12.0-23.0.dev
2020-11-04 dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com Version 2.12.0-22.0.dev

Earlier failed attempt: #22325

2020-11-05 dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com Version 2.12.0-24.0.dev
2020-11-05 dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com Version 2.12.0-23.0.dev
2020-11-04 dart-luci-ci-builder@dart-ci.iam.gserviceaccount.com Version 2.12.0-22.0.dev
@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.

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

Copy link

@leafpetersen leafpetersen left a comment

Choose a reason for hiding this comment

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

LGTM

@iskakaushik iskakaushik added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Nov 5, 2020
@chinmaygarde
Copy link
Contributor

@iskakaushik The //shell/testing/observatory target needs to be updated to fix the presubmit failure.

@iskakaushik
Copy link
Contributor Author

@chinmaygarde yup, have a fix for it, trying to repro locally.

@iskakaushik
Copy link
Contributor Author

I'm not sure if ac35df0 follows are the right language paradigms, but it passes the check locally.

@fluttergithubbot fluttergithubbot merged commit f3c5ecd into flutter:master Nov 5, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 6, 2020
chaselatta pushed a commit to chaselatta/engine that referenced this pull request Nov 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants