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

chore: Gradle Kotlin DSL #659

Merged
merged 1 commit into from
Mar 29, 2021
Merged

Conversation

misa
Copy link
Contributor

@misa misa commented Mar 20, 2021

Gradle build scripts in Kotlin DSL

build.gradle.kts scripts with identical functionality as the original build.gradle scripts.

Improvements

  • Better Android Studio / IntelliJ IDEA support
  • Autocompletion
  • Navigation to source

Gradle Kotlin DSL documentation

Note

  • I was considering using buildSrc for version definitions and extension functions helpers but it didn’t feel that necessary.
  • No libraries versions updates.


dependencies {
implementation(fileTree(mapOf("dir" to "libs", "include" to listOf("*.jar"))))
// additionalAppDependencies placeholder (don't change or delete this line)
Copy link
Member

Choose a reason for hiding this comment

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

Espresso driver dynamically modifies some placeholders like here by https://github.com/appium/appium-espresso-driver#espresso-build-config , so they also should be updated to this syntax. (Probably add ()?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I fixed the additional dependencies and also I updated the documentation links.

@mykola-mokhnach
Copy link
Contributor

@tinder-ktarasov could you please review this PR and confirm it works for you?

Copy link
Member

@KazuCocoa KazuCocoa left a comment

Choose a reason for hiding this comment

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

Seems good to me...

@mykola-mokhnach mykola-mokhnach merged commit 9612d60 into appium:master Mar 29, 2021
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.

None yet

3 participants