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
Fix adding framework targets to AppClip #2530
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this @sampettersson. Would you mind updating the CHANGELOG?
@all-contributors add @sampettersson for code |
@pepibumur I've put up a pull request to add @sampettersson! π |
@pepibumur I did add it here: https://github.com/tuist/tuist/pull/2530/files#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4edR14 did I do it wrong? π |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! Try to rebase main, there are some unrelated acceptance tests failing. Ping me here if it fails even after rebase but since the main is green, it should work fine.
@@ -11,6 +11,7 @@ Please, check out guidelines: https://keepachangelog.com/en/1.0.0/ | |||
### Changed | |||
|
|||
- Replace `ExpressibleByStringLiteral` with `ExpressibleByStringInterpolation` for `ProjectDescription` objects by [@DimaMishchenko](https://github.com/DimaMishchenko) | |||
- Fix adding framework targets to AppClip by [@sampettersson](https://github.com/sampettersson) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a link to the PR please (see below) => I think the log above does not have a PR link by mistake
9745fa9
to
e35028e
Compare
@all-contributors add @sampettersson for code |
@pepibumur I've put up a pull request to add @sampettersson! π |
Short description π
validLinks
wasn't defined for.appClip
ending in an errorTarget A has a dependency with target B of type framework for platform 'iOS' which is invalid or not supported yet.
this PR resolves that.Checklist β
CHANGELOG.md
has been updated to reflect the changes. In case of a breaking change, it's been flagged as such.