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

Expand use_frameworks! DSL to accept linkage style. #581

Merged
merged 2 commits into from
Aug 22, 2019

Conversation

dnkoutso
Copy link
Contributor

@dnkoutso dnkoutso commented Aug 11, 2019

  • Publish RFC

RFC: CocoaPods/CocoaPods#9099

@dnkoutso dnkoutso added this to the 1.9 milestone Aug 11, 2019
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch from d5f6a30 to a3c06f6 Compare August 11, 2019 11:54
@dnkoutso dnkoutso changed the title Expand use_frameworks! DSL to accept linkage style. [DNM] Expand use_frameworks! DSL to accept linkage style. Aug 11, 2019
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch 9 times, most recently from 8be6999 to e07f8f7 Compare August 11, 2019 15:17
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch from e07f8f7 to 9bafc2f Compare August 11, 2019 21:10
@dnkoutso dnkoutso changed the title [DNM] Expand use_frameworks! DSL to accept linkage style. Expand use_frameworks! DSL to accept linkage style. Aug 11, 2019
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch 7 times, most recently from aef2102 to e910c68 Compare August 19, 2019 10:32
inheritance: search_paths
name: SampleAppTests
name: SampleApp
uses_frameworks: true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is used for a test to ensure we can parse a pre-existing serialized YAML Podfile with a boolean value for uses_frameworks.

@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch from e910c68 to 11e601f Compare August 19, 2019 17:53
@dnkoutso dnkoutso requested review from amorde and paulb777 August 19, 2019 18:04
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch 2 times, most recently from d58ad0d to 3cc6efb Compare August 19, 2019 18:36
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch 4 times, most recently from 8a7bbea to e9372c7 Compare August 21, 2019 09:02
@dnkoutso dnkoutso force-pushed the use_frameworks_linkage_dsl branch from e9372c7 to eb59870 Compare August 21, 2019 16:04
inheritance: search_paths
name: SampleAppTests
name: SampleApp
uses_frameworks:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

likewise this is used to test that we can load uses_frameworks hash value from a serialized YAML Podfile.

@dnkoutso dnkoutso merged commit 71a3230 into CocoaPods:master Aug 22, 2019
@dnkoutso dnkoutso deleted the use_frameworks_linkage_dsl branch August 22, 2019 19:12
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.

3 participants