Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename KMM to KMP #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

EchoEllet
Copy link

@EchoEllet EchoEllet commented Oct 26, 2023

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

@iostyle
Copy link

iostyle commented Dec 2, 2024

kdoctor 1.1.0
Kotlin Multiplatform Mobile Plugin: not installed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants