square / anvil

A Kotlin compiler plugin to make dependency injection with Dagger 2 easier.
Apache License 2.0
1.29k stars 76 forks source link

Fix contributed interface merging when KSP backend is enabled #902

Closed ZacSweers closed 4 months ago

ZacSweers commented 4 months ago

I had assumed that ModuleMergerIr somewhere also handled merging contributed interfaces, but that's actually still done separately in InterfaceMerger. With the KSP controls, we currently don't add this impl if we enable the KSP backend. Easy enough fix for now is to move it back up to before, but this'll be another thing we need to do to fully support K2 (i.e. InterfaceMergerIr).

ZacSweers commented 4 months ago

Superseded by #907