Skip to content

Conversation

blasten
Copy link

@blasten blasten commented Nov 20, 2020

implementation is provided by the Android Gradle plugin. It cannot be called before the Android plugin has loaded, and registered an implementation callback.

afterEvaluate ensures that implementation is only called after this plugin loaded.

@blasten blasten requested review from xster and ditman November 20, 2020 22:19
@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Nov 20, 2020
@google-cla google-cla bot added the cla: yes label Nov 20, 2020
Copy link
Member

@xster xster left a comment

Choose a reason for hiding this comment

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

LGTM. Can we add a test?

@blasten
Copy link
Author

blasten commented Nov 21, 2020

@xster test added

@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • Please get at least one approved review before re-applying this label. Reviewers: If you left a comment approving, please use the "approve" review action instead.
  • The status or check suite Mac build_tests has failed. Please fix the issues identified (or deflake) before re-applying this label.

@blasten blasten merged commit 61dfbfe into flutter:master Nov 21, 2020
@blasten blasten deleted the after_evaluate branch November 21, 2020 02:03
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

This should not be running in the devicelab since it does not require dedicated hardware.

@blasten
Copy link
Author

blasten commented Nov 21, 2020

I discussed adding this to LUCI with @godofredoc. Currently, it's a manual process, and cannot be done in a single PR like this one.

This would have to be done after the holidays most likely.

@jonahwilliams
Copy link
Contributor

I filled #70978 to track. Its not so urgent that we need to deal with it right now

@blasten
Copy link
Author

blasten commented Nov 21, 2020

Thanks @jonahwilliams ! I know @godofredoc is on top of improving this devX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants