Skip to content

Commit a4ad4c2

Browse files
authored
Revise Metro bugs in di.md (#165)
Updated the documentation to reflect the current state of bugs in Metro. Updated section on solved bugs to specify version 2.3.20.
1 parent 7ba7a52 commit a4ad4c2

File tree

1 file changed

+14
-8
lines changed

1 file changed

+14
-8
lines changed

docs/di.md

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -551,27 +551,33 @@ See the documentation for [`Scoped`](scope.md#scoped) for more details.
551551

552552
Metro is in an early stage and there are several bugs blocking a full roll out.
553553

554+
555+
#### Missing integrations
556+
557+
Almost all App Platform specific custom extensions for `kotlin-inject-anvil` were migrated to Metro, including
558+
`@ContributesRenderer` and `@ContributesRobot`. However the integration for `@ContributesRealImpl` and
559+
`@ContributesMockImpl` is missing and still needs to be ported.
560+
561+
### Solved bugs in Kotlin 2.3.20 & Metro 0.10.0. Older versions won't receive the fix.
562+
554563
#### No full KMP support
555564

556565
Metro is ready to support KMP, but targets other than JVM/Android fail to merge types contributed with
557566
`@ContributesTo` and `@ContributesBinding`. App Platform makes heavy use of them. This is called out in the
558-
[Metro documentation](https://zacsweers.github.io/metro/latest/multiplatform.html#multiplatform). There is a chance
567+
[Metro documentation](https://zacsweers.github.io/metro/0.7.7/multiplatform/). There is a chance
559568
this will be fixed in Kotlin 2.3.
560569

561570
> There is one issue in the repo right now where the compiler appears to have a bug with generated FIR declarations where it doesn’t deserialize them correctly on non-JVM targets. Waiting for feedback from JB.
562571
572+
The ticket [metro/460](https://github.com/ZacSweers/metro/issues/460) and related in Kotlin [KT-58886](https://youtrack.jetbrains.com/issue/KT-58886) are closed.
573+
563574
#### Incremental compilation issues
564575

565576
While testing Metro in App Platform, we encountered incremental compilation issues that impacted merging components
566-
and generated wrong code. This ticket is [metro/997](https://github.com/ZacSweers/metro/issues/997).
567-
568-
Other IC issues are reported under [KT-75865](https://youtrack.jetbrains.com/issue/KT-75865).
577+
and generated wrong code. This ticket is [metro/997](https://github.com/ZacSweers/metro/issues/997) (closed).
569578

570-
#### Missing integrations
579+
Other IC issues are reported under [KT-75865](https://youtrack.jetbrains.com/issue/KT-75865) (closed).
571580

572-
Almost all App Platform specific custom extensions for `kotlin-inject-anvil` were migrated to Metro, including
573-
`@ContributesRenderer` and `@ContributesRobot`. However the integration for `@ContributesRealImpl` and
574-
`@ContributesMockImpl` is missing and still needs to be ported.
575581

576582
### Migration
577583

0 commit comments

Comments
 (0)