Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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.

}

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.

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