Skip to content

Xcode 15: fixed release build#3034

Merged
NachoSoto merged 1 commit into
mainfrom
fix-xcode-15-release-build
Aug 17, 2023
Merged

Xcode 15: fixed release build#3034
NachoSoto merged 1 commit into
mainfrom
fix-xcode-15-release-build

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

Fixes #3028 (comment)
This was wrong in #3032 because Purchases.storeKit2Setting is only available in DEBUG builds, but we didn't catch it because we have no CI for Xcode 15 yet, which this adds as well.

Fixes #3028 (comment)
This was wrong in #3032 because `Purchases.storeKit2Setting` is only available in `DEBUG` builds, but we didn't catch it because we have no CI for Xcode 15 yet, which this adds as well.

@joshdholtz joshdholtz left a comment

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.

🚀

@NachoSoto NachoSoto merged commit 91890ed into main Aug 17, 2023
@NachoSoto NachoSoto deleted the fix-xcode-15-release-build branch August 17, 2023 03:44
NachoSoto pushed a commit that referenced this pull request Aug 17, 2023
**This is an automatic release.**

### Bugfixes
* `Xcode 15`: fixed release build (#3034) via NachoSoto (@NachoSoto)
NachoSoto added a commit that referenced this pull request Aug 17, 2023
The `build_tv_watch_mac` job already builds on Release mode (`Carthage`'s default).
This would have also helped detect the issue fixed by #3034.

I've also changed the API tester jobs to compile on release, though those aren't ran by CI.
NachoSoto added a commit that referenced this pull request Aug 22, 2023
The `build_tv_watch_mac` job already builds on Release mode
(`Carthage`'s default).
This would have also helped detect the issue fixed by #3034.

I've also changed the API tester jobs to compile on release, though
those aren't ran by CI.
MarkVillacampa pushed a commit that referenced this pull request Sep 6, 2023
Fixes
#3028 (comment)
This was wrong in #3032 because `Purchases.storeKit2Setting` is only
available in `DEBUG` builds, but we didn't catch it because we have no
CI for Xcode 15 yet, which this adds as well.
MarkVillacampa pushed a commit that referenced this pull request Sep 6, 2023
**This is an automatic release.**

### Bugfixes
* `Xcode 15`: fixed release build (#3034) via NachoSoto (@NachoSoto)
MarkVillacampa pushed a commit that referenced this pull request Sep 6, 2023
The `build_tv_watch_mac` job already builds on Release mode
(`Carthage`'s default).
This would have also helped detect the issue fixed by #3034.

I've also changed the API tester jobs to compile on release, though
those aren't ran by CI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants