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

Inject on a device? #84

Merged
merged 1 commit into from
Jan 18, 2024
Merged

Inject on a device? #84

merged 1 commit into from
Jan 18, 2024

Conversation

johnno1962
Copy link
Collaborator

Recently, sparked by this issue using InjectionIII on a device has been made far easier and more reliable. With this change, Inject should also work on a device if people add a build phase and fall back to the old way if they don't.

@krzysztofzablocki krzysztofzablocki merged commit e91205e into krzysztofzablocki:main Jan 18, 2024
@krzysztofzablocki
Copy link
Owner

🙇

@johnno1962
Copy link
Collaborator Author

Thanks! There are some other tweaks in the HotSwiftUI.swift you might want to check out bringing the "cost" of the @ObserveInjection property wrapper down to zero for a release build in terms of storage.

It seems like @Perjan is up and running now, do you want to add a 1.3.0 tag for the new code? I'll release the xrOS bundle containing version of the app in a few days.

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.

None yet

2 participants