Skip to content

Conversation

@Plamen5kov
Copy link
Contributor

@Plamen5kov Plamen5kov commented Dec 12, 2017

problem
test-app project in android studio is different than a normal user project template, that causes difficult debugging experience

solution
make test-app project structure resemble user project template closely as possible:

  • move android-metadata-generator to test-app/build-tools
    • all input arguments are in .txt files (makes for easier debugging)
  • move android-static-binding-generator to test-app/build-tools
    • all input arguments are in txt files (makes for easyer debugging)
  • move android-dts-generator to test-app/build-tools
  • use build.gradle from project template
    • metadata is generated automatically
    • static binding generator is ran if needed and input directory is test-app/app/src/main/assets/app
  • add dts generator as a submodule

expected behavior
no changes in behavior:
run: ./gradlew on root level to build android tgz
run: ./gradlew runtests to run unit tests

@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

1 similar comment
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/project-structure-refactor branch from 6d243b8 to 8bf52e0 Compare December 18, 2017 09:03
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/project-structure-refactor branch from 8bf52e0 to 7e69283 Compare December 18, 2017 09:16
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

@Natalia-Hristova
Copy link

run ci

@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

@Natalia-Hristova
Copy link

run ci

@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

1 similar comment
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

@Plamen5kov Plamen5kov force-pushed the plamen5kov/project-structure-refactor branch from adf4597 to 60541c3 Compare December 18, 2017 12:55
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💔

@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💚

1 similar comment
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💚

@Plamen5kov Plamen5kov force-pushed the plamen5kov/project-structure-refactor branch from fa2d9e5 to 9643e1e Compare December 18, 2017 15:47
@ns-bot
Copy link

ns-bot commented Dec 18, 2017

💚

@Plamen5kov Plamen5kov force-pushed the plamen5kov/project-structure-refactor branch from 9643e1e to 3c7b72a Compare December 19, 2017 06:55
@ns-bot
Copy link

ns-bot commented Dec 19, 2017

💔

@ns-bot
Copy link

ns-bot commented Dec 19, 2017

💚

@Plamen5kov Plamen5kov merged commit 81f70ad into master Dec 19, 2017
@Plamen5kov Plamen5kov deleted the plamen5kov/project-structure-refactor branch December 19, 2017 07:39
petekanev added a commit that referenced this pull request Jan 4, 2018
Plamen5kov added a commit that referenced this pull request Jan 5, 2018
* Revert "refactor(project template): replace runtime project template with user template (#891)"

This reverts commit 81f70ad.

* fix(tooling): wait for emulator before running adb commands

* chore: update messages
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.

5 participants