Architecture-samples: New sample: Kotlin version of todo-mvp

Created on 21 Nov 2016  路  5Comments  路  Source: android/architecture-samples

Although is not an architectural pattern, I think it could be nice to have a Kotlin version for some of the variants. Interesting points would be the amount of code and size of the app.

I personally started the todo-mvp-kotlin in my forked repo as an example. so far I converted:

  • addedittask
  • data

Please note that this is still in progress :) and feel free to contribute to it if you're interested!

https://github.com/Shyish/android-architecture/tree/dev-todo-mvp-kotlin

New sample community request

Most helpful comment

Looking forward to comparing APK size, LOC and method count. Please make sure to mention in the README that Kotlin is not officially supported.

All 5 comments

Looking forward to comparing APK size, LOC and method count. Please make sure to mention in the README that Kotlin is not officially supported.

So, first version is done, although there could be some more optimizations to be made.

https://github.com/Shyish/android-architecture/tree/dev-todo-mvp-kotlin

@JoseAlcerreca @malmstein Please take a look when you can and let me know what else do you need :)

Hey @Shyish I was also curious how the app would look written in Kotlin and I started: https://github.com/LordRaydenMK/android-architecture/tree/todo-mvp-kotlin

Looks like you are way ahead of me. I am almost done with the data package.

I noticed some areas in the code on your fork that could be improved. I would like to discuss them with you and maybe contribute with a pull request. Let me know what you think.

Hi @LordRaydenMK, sounds great to me! You can just create an issue for discussion or directly a PR in my repo and we can discuss it there :)

I'm promoting #254 to the main list since it's already finished. Closing this but if you want to incorporate changes from your forks to @SerjSmor's I'm sure it'll be appreciated.

Was this page helpful?
0 / 5 - 0 ratings