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

[AUTOMATIC] Release/4.12.0 #1890

Merged
merged 3 commits into from
Sep 8, 2022
Merged

[AUTOMATIC] Release/4.12.0 #1890

merged 3 commits into from
Sep 8, 2022

Conversation

revenuecat-ops
Copy link
Collaborator

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

This is an automatic release.

Bugfixes

New Features

Other Changes

@NachoSoto
Copy link
Contributor

NachoSoto commented Sep 7, 2022

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

@NachoSoto
Copy link
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