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

Extract the Kotlin Mutliplatform version to a seperate repository #42

Closed
Skaldebane opened this issue Aug 13, 2022 · 9 comments
Closed

Comments

@Skaldebane
Copy link
Contributor

Skaldebane commented Aug 13, 2022

Asslamu Alaikum! I just came across this library through one @ahmedre's talks, yet had a hard time finding it here (there's nothing about it in the main Adhan repo), because it's a branch in this repo.

I guess it would be better if it was separated into its own library, simply called adhan-kotlin (many people may just want to use the Kotlin version on Android without caring for KMP, so this name would make more sense for them).

Thanks!

@Skaldebane
Copy link
Contributor Author

Any updates on this?

@ahmedre
Copy link
Collaborator

ahmedre commented May 16, 2023

wa3laikum alsalam,
no update - still not sure whether we'll do this or keep them as part of the same repo. @z3bi / @sgtsquiggs any strong opinions here? I don't mind maintaining both versions but there's little reason to keep maintaining a Java version if a Kotlin version exists (considering it also compiles down to the JVM).

if we keep it in the same repo, we could switch to having the main branch be the kmp branch, which would then have a link to the old branch.
walsalam 3alaikum

@sgtsquiggs
Copy link
Contributor

I would say take the option with the least ongoing maintenance. We can update documentation to make it easier to find if that is a problem.

@Skaldebane
Copy link
Contributor Author

I agree with @ahmedre, maintaining both the Java and Kotlin libraries is generally repeated work since the Kotlin version can work perfectly with Java (on the JVM).
I think it's a good way forward as it will reduce the maintenance cost of having to keep two versions of the same thing in sync.

@GulmirzaNorboyev
Copy link

Assalomu alaykum! I think it's better to use the Kotlin DateTime API, not java.util.Calendar, for the library to work properly for KMM, to work on both platforms...

@ahmedre
Copy link
Collaborator

ahmedre commented Jun 13, 2023

wa3laikum alsalam -
we are in the kotlin_multiplatform branch since otherwise it wouldn't work with Kotlin Multiplatform.
walsalam 3alaikum.

@ahmedre
Copy link
Collaborator

ahmedre commented Nov 13, 2023

salam 3alaikum,
we've made the KMP version the primary one since the KMP version works on Java targets.
we won't be splitting the repositories at this time.
walsalam 3alaikum.

@ahmedre ahmedre closed this as completed Nov 13, 2023
@Skaldebane
Copy link
Contributor Author

Jazakallah brother, that's the best way forward.

@ahmedre
Copy link
Collaborator

ahmedre commented Nov 14, 2023

waiyyakum!

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

4 participants