Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

delete all example/android/app/gradle.properties files #1293

Merged
merged 2 commits into from
Oct 23, 2019

Conversation

amirh
Copy link
Contributor

@amirh amirh commented Oct 21, 2019

Description

Following @blasten investigation these files are not needed, and may create some AndroidX related issues.

These files were introduced in: flutter/plugins#1103

Copy link
Contributor

@collinjackson collinjackson left a comment

Choose a reason for hiding this comment

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

LGTM once the tests pass. It seems the firebase_analytics plugin is still having build issues.

@collinjackson
Copy link
Contributor

I'm going to go ahead and merge this because it seems like the tests pass and I want to fix the CI.

@collinjackson collinjackson merged commit 31cad01 into firebase:master Oct 23, 2019
@collinjackson
Copy link
Contributor

kroikie pushed a commit to collinjackson/flutterfire that referenced this pull request Nov 15, 2019
* delete all example/android/app/gradle.properties files
@firebase firebase locked and limited conversation to collaborators Aug 8, 2020
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.

3 participants