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
Xcode 13 Support #3486
Xcode 13 Support #3486
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As the lib also supports running on Linux and Windows, we need to be broader in the #if
directive to embrace those systems in the verification
Source/Combine.swift
Outdated
@@ -22,7 +22,7 @@ | |||
// THE SOFTWARE. | |||
// | |||
|
|||
#if canImport(Combine) | |||
#if !(os(iOS) && (arch(i386) || arch(arm))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this check must be broader to check for windows and linux that doesn't support Combine framework
Tests/CombineTests.swift
Outdated
@@ -22,7 +22,7 @@ | |||
// THE SOFTWARE. | |||
// | |||
|
|||
#if canImport(Combine) | |||
#if !(os(iOS) && (arch(i386) || arch(arm))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems great now, thanks
* Update for Xcode 13, workaround Combine issues. * Disable Combine on Windows and Linux.
Issue Link π
#3481
Goals β½
This PR tracks changes necessary for Xcode 13. It also adds a workaround for Combine-related build issues by limiting Combine features to 64-bit platforms as well as the ability to import Combine.
Implementation Details π§
No functional changes, just deployment targets.
Testing Details π
No new tests.