Skip to content

Paywalls: RevenueCatUI CocoaPods support#3368

Merged
NachoSoto merged 5 commits into
mainfrom
paywalls-cocoapods
Nov 1, 2023
Merged

Paywalls: RevenueCatUI CocoaPods support#3368
NachoSoto merged 5 commits into
mainfrom
paywalls-cocoapods

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

Fixes #3152.

@NachoSoto NachoSoto requested a review from a team October 31, 2023 22:49
name = SPMInstallation;
packageProductDependencies = (
B3A9296026E2AAFB00C5E699 /* RevenueCat */,
4F3810982AF1B98200441136 /* RevenueCatUI */,

@NachoSoto NachoSoto Oct 31, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

SPMInstallation will now test this as well.

Comment thread fastlane/Fastfile Outdated
end

def push_revenuecatui_pod
pod_push(path: "RevenueCatUI.podspec", synchronous: true)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we technically don't need synchronous for this one, but it can't hurt to have it

Comment thread .circleci/config.yml
Comment on lines +721 to +732
command: bundle exec fastlane push_revenuecat_pod
no_output_timeout: 30m

push-revenuecatui-pod:
<<: *base-job
steps:
- checkout
- install-bundle-dependencies
- trust-github-key
- run:
name: Deploy new version
command: bundle exec fastlane push_revenuecatui_pod

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why not move this to the fastlane level?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You mean combining both?
So we can retry each individually if any of them fail without having to push code to do that.

@NachoSoto NachoSoto force-pushed the paywalls-cocoapods branch 2 times, most recently from 0cdebb6 to f343836 Compare November 1, 2023 20:07
Comment thread .circleci/config.yml
<<: *release-branches
- installation-tests-cocoapods:
xcode_version: '14.3.0'
<<: *release-branches

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

⚠️ remove commit after testing.

@NachoSoto NachoSoto force-pushed the paywalls-cocoapods branch 2 times, most recently from 18eaba4 to 458c3a8 Compare November 1, 2023 22:31
@NachoSoto NachoSoto enabled auto-merge (squash) November 1, 2023 22:33
@NachoSoto NachoSoto merged commit a465e99 into main Nov 1, 2023
@NachoSoto NachoSoto deleted the paywalls-cocoapods branch November 1, 2023 22:51
NachoSoto pushed a commit that referenced this pull request Nov 2, 2023
**This is an automatic release.**

### RevenueCatUI
* `Paywalls`: `RevenueCatUI` CocoaPods support (#3368) via NachoSoto
(@NachoSoto)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RevenueCatUI not available when installing 4.26.0-beta.2 using cocopods

3 participants