Skip to content

Upgrade to Gradle 4.8 #1350

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

Merged
merged 6 commits into from
Jun 12, 2018
Merged

Upgrade to Gradle 4.8 #1350

merged 6 commits into from
Jun 12, 2018

Conversation

SUPERCILEX
Copy link
Collaborator

No description provided.

Signed-off-by: Alex Saveau <[email protected]>
Signed-off-by: Alex Saveau <[email protected]>
Signed-off-by: Alex Saveau <[email protected]>
Signed-off-by: Alex Saveau <[email protected]>
@SUPERCILEX SUPERCILEX requested a review from samtstern as a code owner June 11, 2018 19:59
organizationUrl.set("https://firebase.google.com")
roles.set(listOf("Project-Administrator", "Developer"))
timezone.set("-8")
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@lsirac If you can send me a developer DSL like this, I'll add you to the list too. 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

BTW @lsirac totally optional, if you don't want to expose your email address or name you can skip.

@@ -11,11 +11,11 @@ object Config {
}

object Plugins {
const val android = "com.android.tools.build:gradle:3.2.0-alpha15"
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't even notice that we had moved to alpha in the gradle --> kotlin switch. Is there anything more stable that will work for us?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That was for lint to work. Also, we already shipped v4.0.1 without any problems so I wouldn't worry about it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ohhhhhhhhh, but I forgot that it will make it hard for people to use the sample 😕. Well unless we want to undo a few weeks of PRs, there's no going back. Sorry about that! 😊

Copy link
Contributor

Choose a reason for hiding this comment

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

NP let's just get onto a beta-or-better train as soon as one is available.

@@ -1,5 +1,5 @@
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-4.7-all.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-4.8-all.zip
Copy link
Contributor

Choose a reason for hiding this comment

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

So has the issue we had with 4.8 been fixed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yup, in bintray 1.8.1.

organizationUrl.set("https://firebase.google.com")
roles.set(listOf("Project-Administrator", "Developer"))
timezone.set("-8")
}
Copy link
Contributor

Choose a reason for hiding this comment

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

BTW @lsirac totally optional, if you don't want to expose your email address or name you can skip.

@samtstern
Copy link
Contributor

I am going to merge and let @lsirac send a PR later for adding his credits to the POM (completely optional)

@samtstern samtstern merged commit 5fac07e into firebase:version-4.1.0-dev Jun 12, 2018
@samtstern samtstern added this to the 4.1.0 milestone Jun 12, 2018
@SUPERCILEX SUPERCILEX deleted the gradle-4.8 branch June 12, 2018 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants