Skip to content

[flutter_tools] remove globals from android_workflow#69548

Merged
jonahwilliams merged 3 commits intoflutter:masterfrom
jonahwilliams:android_workflow_globals
Nov 6, 2020
Merged

[flutter_tools] remove globals from android_workflow#69548
jonahwilliams merged 3 commits intoflutter:masterfrom
jonahwilliams:android_workflow_globals

Conversation

@jonahwilliams
Copy link
Contributor

@jonahwilliams jonahwilliams commented Nov 2, 2020

Description

Remove globals from android_workflow.dart, specifically the license validator. Makes a static method an instance method to make it easier to inject dependencies into.

#47161

@flutter-dashboard flutter-dashboard bot added tool Affects the "flutter" command-line tool. See also t: labels. work in progress; do not review labels Nov 2, 2020
@google-cla google-cla bot added the cla: yes label Nov 2, 2020
@jonahwilliams jonahwilliams marked this pull request as ready for review November 2, 2020 15:50
Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@jonahwilliams jonahwilliams merged commit e18533e into flutter:master Nov 6, 2020
@jonahwilliams jonahwilliams deleted the android_workflow_globals branch November 6, 2020 18:55
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.

2 participants