Skip to content

Conversation

@blasten
Copy link

@blasten blasten commented Jul 22, 2021

This code was introduced to workaround AndroidX compatibilities issues back in 2019.

Related to #86802

@blasten blasten added c: contributor-productivity Team-specific productivity, code health, technical debt. platform-android Android applications specifically labels Jul 22, 2021
@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Jul 22, 2021
@google-cla google-cla bot added the cla: yes label Jul 22, 2021
@blasten blasten requested review from jmagman and jonahwilliams July 22, 2021 23:41
@bernaferrari
Copy link
Contributor

Should you also remove

android.useAndroidX=true
android.enableJetifier=true

from gradle.properties? At least flutter create adds them.

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.

LGTM

I believe there are some additional androidX checks as well as analytics in the code base. Should we remove all of it, or just the aar workaround here?

@blasten
Copy link
Author

blasten commented Jul 23, 2021

Yup. This is mostly around the workarounds. The AndroidX vs non AndroidX distinction can also be removed. I will do that in the next PR.

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

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. platform-android Android applications specifically 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