Skip to content

Add support for multiplatform targets#2

Open
asnaeb wants to merge 6 commits into
iamcalledrob:masterfrom
asnaeb:master
Open

Add support for multiplatform targets#2
asnaeb wants to merge 6 commits into
iamcalledrob:masterfrom
asnaeb:master

Conversation

@asnaeb

@asnaeb asnaeb commented Sep 4, 2025

Copy link
Copy Markdown

graphics-shapes is now multiplatform and this PR adds support for the missing multiplatform targets. Kotlin version and dependencies have been updated as well.

@iamcalledrob iamcalledrob left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, cool to see the underlying library is now multiplatform.

Unfortunately I don't have a Mac handy that can build iOS+Mac artifacts to publish to maven central for non-android/JVM.

Feel free to set up GitHub actions to generate artifacts if you'd like to see another release cut.

I wish there was a better way of handling this problem, it's such a sore point in Kotlin multiplatform.

Comment thread build.gradle.kts
id("org.jetbrains.compose") version "1.6.11"
id("com.android.library") version "8.2.0"
kotlin("multiplatform") version "2.2.10"
id("org.jetbrains.kotlin.plugin.compose") version "2.2.10"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious why the plugin is needed?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@asnaeb

asnaeb commented Sep 5, 2025

Copy link
Copy Markdown
Author

I have setup github actions. You'll need to set up the proper environment variables, then just publish a release on github and the workflow should start

Comment thread build.gradle.kts
// - signing.gnupg.secretKeyRingFile
// - mavenCentralUsername
// - mavenCentralPassword

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not apply anymore now that the publishing happens via CI/CD

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