Skip to content

[AUTOMATIC] Release/4.12.0#1890

Merged
NachoSoto merged 3 commits into
mainfrom
release/4.12.0
Sep 8, 2022
Merged

[AUTOMATIC] Release/4.12.0#1890
NachoSoto merged 3 commits into
mainfrom
release/4.12.0

Conversation

@revenuecat-ops

@revenuecat-ops revenuecat-ops commented Sep 7, 2022

Copy link
Copy Markdown
Collaborator

This is an automatic release.

Bugfixes

New Features

Other Changes

Comment thread Tests/InstallationTests/CommonFiles/RevenueCat-Swift.h Outdated
@NachoSoto

NachoSoto commented Sep 7, 2022

Copy link
Copy Markdown
Contributor

Looks like Carthage tests are crashing the simulator for some reason (ran them 3 times).

@NachoSoto

Copy link
Copy Markdown
Contributor

I got Carthage tests passing locally with no issues... but I did fix a warning by removing the unnecessary FRAMEWORK_SEARCH_PATHS.

NachoSoto added a commit that referenced this pull request Sep 8, 2022
For #1890.

The failure was likely because the simulators had a dialog which was blocking the tests from continuing. In order to fix that, I removed the test host from the target, which was never necessary and will make running these tests a bit faster.

### Changes:
- `CarthageInstallation`: removed unnecessary `FRAMEWORK_SEARCH_PATHS`
- Removed test host
- Added single scheme with `test` action
@NachoSoto NachoSoto requested a review from a team September 8, 2022 20:21
@NachoSoto NachoSoto merged commit 51f4834 into main Sep 8, 2022
@NachoSoto NachoSoto deleted the release/4.12.0 branch September 8, 2022 21:11
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.

3 participants