kdoctor icon indicating copy to clipboard operation
kdoctor copied to clipboard

Rename KMM to KMP

Open EchoEllet opened this issue 2 years ago • 6 comments

Since Kotlin Multiplatform Mobile (KMM) has been renamed to Kotlin Multiplatform (KMP) source

I renamed everything from KMM and Kotlin multiplatform mobile to KMP and Kotlin multiplatform without affecting the logic, just labeling that is all

because we can't rename everything since the plugin in Android studio is still called KMM so I didn't rename it, yet

Example of running in a successful state:

Before

./kdoctor 
Environment diagnose (to see all details, use -v option):
[✓] Operation System
[✓] Java
[✓] Android Studio
[✓] Xcode
[✓] CocoaPods

Conclusion:
  ✓ Your operation system is ready for Kotlin Multiplatform Mobile Development!

After

./kdoctor
Environment diagnose (to see all details, use -v option):
[✓] Operation System
[✓] Java
[✓] Android Studio
[✓] Xcode
[✓] CocoaPods

Conclusion:
  ✓ Your operation system is ready for Kotlin Multiplatform Development!

I also updated the README.md and the tests and the other messages

I also change one additional thing which is not related to the subject

Before

Shell.values().firstOrNull { it.path == shellPath }

After

Shell.entries.firstOrNull { it.path == shellPath }

Since in Kotlin 1.9.0, it's recommended to use entries instead of values() in enum classes

there is one thing but I didn't change it the diagnose should be diagnosis

it doesn't make much difference at least to me so I didn't update it

EchoEllet avatar Oct 26 '23 01:10 EchoEllet

kdoctor 1.1.0 Kotlin Multiplatform Mobile Plugin: not installed

iostyle avatar Dec 02 '24 10:12 iostyle

We need this PR to be merged. Since KMP plugin has been renamed, KDoctor always fails to find the plugin attached to the Android Studio

carloshmartins avatar Jan 31 '25 21:01 carloshmartins

Hello everyone,

This bug is still affecting all installations of kdoctor that use the current plugin. I strongly encourage merging this PR.

needToRoll avatar Mar 01 '25 22:03 needToRoll

@terrakok can this be merged for the love of programming!

KDoctor doesn't recognise KMM plugin in android studio. Screenshot 2025-03-10 at 08 13 49

algoflows avatar Mar 10 '25 08:03 algoflows

How long till we get a fix for this?

tarannum22 avatar Apr 10 '25 14:04 tarannum22

We need this PR to be merged.

For backward compatibility, I didn't rename the plugin id from kmm to kmp at the time of this PR and left a TODO since it will break eventually.

Now the plugin id has been renamed, we should update this PR to use kmp and remove the TODO.

EchoEllet avatar Apr 10 '25 16:04 EchoEllet

Still not merged, 😭

image

byaruhaf avatar Aug 09 '25 01:08 byaruhaf

I don't think you will need kdoctor anymore. The functionality of kdoctot is now built-in as part of the KMP plugin.

EchoEllet avatar Aug 09 '25 14:08 EchoEllet