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

Update dependencies for library and konvert to Kotlin for readability #2

Closed
LukasAnda opened this issue Sep 19, 2018 · 4 comments
Closed

Comments

@LukasAnda
Copy link

Do you want to request a feature or report a bug?
Feature
What is the current behaviour?
As expected, but the code is quite long and can be simplified
If the current behaviour is a bug, please provide the steps to reproduce.

Any logs, error output, bug reports etc?

What is the expected behaviour?

Any other comments?

I can create a pull request in few weeks so you can merge it

What versions of software are you using?

  • Device Information:

  • Android Version:

  • Configuration Information:

  • Misc:

@nisrulz
Copy link
Owner

nisrulz commented Sep 19, 2018

As an app, it makes sense to convert to Kotlin but as a library, it doesn't. Converting to Kotlin only means that now the library would have a transitive dependency on the Kotlin standard library. For people who have yet to move to Kotlin in their app's the library if added will only add bloat to their app.

On the other hand, this being in Java is completely useable in Kotlin world. However in the future when there are enough users of Kotlin on Android the library could be ported Kotlin. Until then I do not see any benefit in porting this to Kotlin.

Closing as this proposal doesn't bring any useful benefit to users of this library.

@nisrulz nisrulz closed this as completed Sep 19, 2018
@LukasAnda
Copy link
Author

Okay, no problem. Also, what about updating the dependencies to the latest version ? The support libraries have moved to androidx library

@nisrulz
Copy link
Owner

nisrulz commented Sep 20, 2018

Packagehunter does not depend on any support library 😄

Take a look at the dependencies block here

@LukasAnda
Copy link
Author

Oh my fault, saw those dependencies for sample app :D

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

No branches or pull requests

2 participants