Skip to content

Removes swiftPMImport from cinterop package name#6803

Merged
JayShortway merged 1 commit into
mainfrom
cinterop-packagename
May 20, 2026
Merged

Removes swiftPMImport from cinterop package name#6803
JayShortway merged 1 commit into
mainfrom
cinterop-packagename

Conversation

@JayShortway

@JayShortway JayShortway commented May 15, 2026

Copy link
Copy Markdown
Member

Motivation

The swiftPMImport prefix was added initially because it would allow us to easily switch to KMP's own experimental SPM integration. Now that we're not doing that, it's no longer necessary. This PR cleans it up. It's a fully internal change.

Note

This PR should be released in the same version as RevenueCat/purchases-kmp#854.


Note

Low Risk
Low risk: a single internal string change to the Compose Resources bundle lookup; main risk is missing resources at runtime if the bundle directory name doesn’t match the packaged output for certain build configurations.

Overview
Updates the COMPOSE_RESOURCES bundle lookup in Bundle+Extensions.swift to remove the swiftPMImport. prefix from the Compose Resources directory name, so RevenueCatUI loads resources from .../composeResources/com.revenuecat.purchases.kn.ui.resources/files instead of the old path.

Reviewed by Cursor Bugbot for commit f3b3f47. Bugbot is set up for automated code reviews on this repo. Configure here.

@JayShortway JayShortway self-assigned this May 15, 2026
@JayShortway JayShortway marked this pull request as ready for review May 15, 2026 09:57
@JayShortway JayShortway requested a review from a team as a code owner May 15, 2026 09:57
@JayShortway JayShortway merged commit 28c37ca into main May 20, 2026
18 of 20 checks passed
@JayShortway JayShortway deleted the cinterop-packagename branch May 20, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants