Skip to content

WIP: [flutter_tools] settings.gradle parsing and validation - #76328

Closed
GaryQian wants to merge 2 commits into
flutter:masterfrom
GaryQian:splitsettings
Closed

GaryQian wants to merge 2 commits into
flutter:masterfrom
GaryQian:splitsettings

Conversation

@GaryQian

Copy link
Copy Markdown
Contributor

Split off of #75739 due to robustness concerns.

Implements parsing of the android/settings.gradle and checks for the correct entries under includes.

@GaryQian GaryQian added tool Affects the "flutter" command-line tool. See also t: labels. a: build Building flutter applications with the tool labels Feb 18, 2021
@google-cla google-cla Bot added the cla: yes label Feb 18, 2021
@GaryQian

Copy link
Copy Markdown
Contributor Author

Turns out that gradle will actually produce a very easy to parse error enumerating the missing imports that this PR computes. This means that this validator feature is not very necessary right now, though the benefit of this is that it is unified with the rest of the validator rather than depending on the gradle error output.

@GaryQian GaryQian closed this Mar 25, 2021
@GaryQian

Copy link
Copy Markdown
Contributor Author

We can reconsider adding this in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: build Building flutter applications with the tool 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