-
Notifications
You must be signed in to change notification settings - Fork 127
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
Thanks for series ! π¨π»βπ» here is quick suggestion #1
Comments
Thank you very much for the tips π |
Is this repo open for contributions? I can open some PR or put some suggestions that you can use in the future projects? |
I don't want to change the code of this repo because it would break the tutorial but suggestions are more than welcome! |
comye1
added a commit
to comye1/MVVMTodo
that referenced
this issue
Apr 29, 2021
comye1
added a commit
to comye1/MVVMTodo
that referenced
this issue
Apr 29, 2021
comye1
added a commit
to comye1/MVVMTodo
that referenced
this issue
Apr 29, 2021
This reverts commit 2136eac.
comye1
added a commit
to comye1/MVVMTodo
that referenced
this issue
Apr 29, 2021
comye1
added a commit
to comye1/MVVMTodo
that referenced
this issue
Apr 29, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, @codinginflow Nice tutorial series,
Just a suggestion you can reduce some nesting using this
with
thanapply
[Edit1]
Nested scoped extension chaining is are considered not a good practice, checkout some awesome kotlin tips be from
Huyen
talkKotlinConf 2019
youtube.com/watch?v=YeqGfKmJM_gto
The text was updated successfully, but these errors were encountered: