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

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Mar 21, 2020

…its) (#17241)"

This reverts commit 047c03b.

This roll also breaks web tests with:

org-dartlang-app:///test/material/icon_button_test.dart:15:7: Warning: Implementing 'Function' is deprecated.
Try removing 'Function' from the 'implements' clause.
class MockOnPressedFunction implements Function {

TBR= @bkonyi

…ter#17241)"

This reverts commit 047c03b.

This roll also breaks web tests with:

```
org-dartlang-app:///test/material/icon_button_test.dart:15:7: Warning: Implementing 'Function' is deprecated.
Try removing 'Function' from the 'implements' clause.
class MockOnPressedFunction implements Function {
```
@yjbanov yjbanov requested a review from bkonyi March 21, 2020 03:18
@yjbanov
Copy link
Contributor Author

yjbanov commented Mar 21, 2020

@bkonyi Looking at #17241 the roll seems to have ignored a test failure and merged anyway:

dart roll 047c ignored errors

Is something on our end misconfigured?

@yjbanov
Copy link
Contributor Author

yjbanov commented Mar 21, 2020

Landing to fix the tree.

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.

2 participants