Good morning. I’m converting my app to Koin annotations ( 3.2.2 + 1.0.3 ) and I think I found one ( ...

# koin

[Davide Giuseppe Farella]

Good morning. I’m converting my app to Koin annotations ( 3.2.2 + 1.0.3 ) and I think I found one ( two? ) issue(s).

After converting all my modules, I have an “aggregator”, like

@Module(includes = [...])
class MyAppModule {

  // couple of defs here
}

And here’s the first issue: the generated code has syntax issues, like includes(.SomeModule ( note the . before the name and missing qualifier )

Since in the includes = [...] I had about 50 modules, I thought I broke the processor, so I grouped them into other modules like

@Module(includes = [A::class, B::class])
class MyAppModule {

  // couple of defs here
}

@Module(includes = [...])
class A

@Module(includes = [...])
class B

But seems like A and B are not processed, as there is generated code only for MyAppModule

[Davide Giuseppe Farella]

message has been deleted

[arnaud.giuliani]

intersting, do you use package name or not?

[arnaud.giuliani]

just to understand the “.SomeModule”

[Davide Giuseppe Farella]

Yes, it is cinescout.database.DatabaseModule .
I found a related issues on the repo, let me find it back

[Davide Giuseppe Farella]

https://github.com/InsertKoinIO/koin-annotations/issues/18

[arnaud.giuliani]

hm 🤔 surprising ... I need to dig into it

[Davide Giuseppe Farella]

Keep me posted 🙏

[arnaud.giuliani]

sure 👍

[Davide Giuseppe Farella]

Thank you for looking into in, Arnaud! Just saw the issue update on GitHub 🍻

[arnaud.giuliani]

cool 👌

[arnaud.giuliani]

I just fixed it. Wait for KA 1.1.0 👍