-
Notifications
You must be signed in to change notification settings - Fork 39
feat: add support for defining Info.plist files using a variable #184
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
Conversation
|
@pepicrft any chance you can take a look please? |
pepicrft
left a comment
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 @alexmx for contributing this one. I left a small comment.
|
@pepicrft any chance you can take a loot at this again please? |
|
@all-contributors add @alexmx for code |
|
I've put up a pull request to add @alexmx! 🎉 |
|
@alexmx thanks a lot for this improvement. I made you a collaborator of the repository so that you can push your code contributions directly to this repository without having to work from forks. |
This PR implements the support for variable InfoPlist file paths for build settings that mirrors the existing API for setting variable entitlements file paths. The issue has been described in detail here: tuist/tuist#6727
This change is needed to unblock tuist/tuist#7404