Make the commonJs source set depend on commonMain

evant / kotlin-inject

Dependency injection lib for kotlin

Apache License 2.0

1.29k stars 60 forks source link

Make the commonJs source set depend on commonMain #374

Closed eygraber closed 7 months ago

evant commented 7 months ago

Interesting, now I'm wondering if anything was broken with those targets before?

eygraber commented 7 months ago

I think it was fine before because common could read platform sources. That changed in Kotlin 2.0 but I figured this could be done before that (and it would technically be correct), so why not get it in ASAP.