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

Add support for StoreKitConfigurationFileReference #573

Merged

Conversation

jcolicchio
Copy link
Contributor

@jcolicchio jcolicchio commented Oct 8, 2020

Addresses #574
Will allow XcodeGen to specify a StoreKit configuration file in the scheme

Short description 📝

Adds a new XML element, StoreKitConfigurationFileReference, to the LaunchAction

Solution 📦

Created StoreKitConfigurationFileReference and added it to LaunchAction following a similar pattern to locationScenarioReference

Implementation 👩‍💻👨‍💻

  • Added class StoreKitConfigurationFileReference mirroring structure of LocationScenarioReference
  • Added property to LaunchAction mirroring all usages and initializations of locationScenarioReference
  • Updated example scheme and ran tests to make sure it still parses

@jcolicchio
Copy link
Contributor Author

@pepibumur I think this is ready to review! Changelog check is failing but I'm not sure if it's related :x

@@ -0,0 +1,43 @@
//
Copy link
Contributor

Choose a reason for hiding this comment

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

@jcolicchio would you mind removing the headers to be consistent with the rest of the codebase?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevermind, I'll merge and fix those before releasing the next version.

@pepicrft pepicrft merged commit d168083 into tuist:master Oct 9, 2020
@jcolicchio jcolicchio deleted the feature/StoreKitConfigurationFileReference branch October 9, 2020 16:14
@pepicrft
Copy link
Contributor

@all-contributors add @jcolicchio for ideas

@allcontributors
Copy link
Contributor

@pepibumur

I've put up a pull request to add @jcolicchio! 🎉

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