Skip to content

feat: add the App Clip's URL to the Run action #185

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

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

Conversation

ns-vasilev
Copy link

No description provided.

@ns-vasilev ns-vasilev force-pushed the feature/app-clips branch from f987e8e to 6bb7627 Compare May 5, 2025 09:13
@ns-vasilev
Copy link
Author

ns-vasilev commented Jun 5, 2025

@pepicrft @fortmarek Hi folks! 👋

Would appreciate it if you could take a look at this PR when you have a chance
I’m currently unable to add App Clips support as described in issue #7036.

Any feedback or a merge would be greatly appreciated 🙏

Copy link
Member

@fortmarek fortmarek left a comment

Choose a reason for hiding this comment

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

What do you think of using URL and only converting the type to String during generation?

@@ -17,6 +17,7 @@ public struct RunAction: Equatable, Codable, Sendable {
public let metalOptions: MetalOptions?
public let expandVariableFromTarget: TargetReference?
public let launchStyle: LaunchStyle
public let appClipInvocationURLString: String?
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public let appClipInvocationURLString: String?
public let appClipInvocationURL: URL?

@fortmarek
Copy link
Member

Would appreciate it if you could take a look at this PR when you have a chance

sorry, this has fallen through the cracks. Feel free to ping us sooner if we forget about something 🙏

@fortmarek
Copy link
Member

@ns-vasilev what do you think about preparing a PR in tuist/tuist that uses the changes from here? You can reference a fork in the Package.swift. It usually helps to make better decision about the XcodeGraph API.

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