Skip to content

Plamen5kov/update project template #875

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 12 commits into from
Nov 8, 2017

Conversation

Plamen5kov
Copy link
Contributor

@Plamen5kov Plamen5kov commented Nov 2, 2017

Do not merge before it's tested with functional tests. Needs QA approval before a merge.

  • Updates the project template to use gradle 4.1 and android plugin for gradle 2.3.3.
  • Integration with Android Studio (you can now open the <app_name>platforms/android project in Android Studio (>=2.3.3))

Stil need to figure out how to debug runtime in Android Studio, when the user project has flavors. (may be done in another PR)
Stil need to update extraction of all possible .jar classes from .aar dependencies! Will do it in the same PR.

related issue: #876
related PR: NativeScript/nativescript-dev-webpack#310
NativeScript/nativescript-cli#3193

move xml files
remove unused dependency from test-app
remove unnecessary folder from build tools
updated gradle to 4.1
updated settings.gradle
updated gradle.properties
@Plamen5kov Plamen5kov requested a review from petekanev November 2, 2017 11:59
@Plamen5kov Plamen5kov self-assigned this Nov 2, 2017
@ns-bot
Copy link

ns-bot commented Nov 2, 2017

💔

@ns-bot
Copy link

ns-bot commented Nov 2, 2017

💔

@petekanev
Copy link
Contributor

petekanev commented Nov 2, 2017

@Plamen5kov Why don't we try to update to Gradle 4.3 and Android Plugin 3.0.0 for improved build speeds?

Edit: never mind, just noticed the changes to the automatic variants matching - https://developer.android.com/studio/build/gradle-plugin-3-0-0-migration.html#variant_aware which we most likely break by the improper use of flavors we have for handling nativescript plugins.

@Plamen5kov
Copy link
Contributor Author

@Pip3r4o once we make the integration with Android Studio, the update from version to version should become automatic. Android Studio should prompt for gradle update once a newer version is installed.

@ns-bot
Copy link

ns-bot commented Nov 3, 2017

💔

@Plamen5kov Plamen5kov requested a review from dtopuzov November 6, 2017 11:23
@ns-bot
Copy link

ns-bot commented Nov 7, 2017

💔

@ns-bot
Copy link

ns-bot commented Nov 7, 2017

💔

@NativeScript NativeScript deleted a comment from Plamen5kov Nov 8, 2017
@NativeScript NativeScript deleted a comment from ns-bot Nov 8, 2017
@dtopuzov
Copy link
Contributor

dtopuzov commented Nov 8, 2017

run ci

@ns-bot
Copy link

ns-bot commented Nov 8, 2017

💚

@Plamen5kov Plamen5kov merged commit 58007be into master Nov 8, 2017
@Plamen5kov Plamen5kov deleted the plamen5kov/update-project-template branch November 8, 2017 09:49
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.

4 participants