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

Feature/strict concurrency split #51

Merged

Conversation

gazreese
Copy link
Contributor

@gazreese gazreese commented Apr 25, 2024

This PR supersedes and includes #48.

This PR updates the above by also updating the Cocoapods functionality to match the Swift Package Manager changes above: 6b5dc80

The changes here can look complicated, but the main change from a functionality perspective is the introduction 'existential any'. There's a good article here - https://www.avanderlee.com/swift/existential-any/.

Type of Change
✨ New feature (non-breaking change which adds functionality)
🛠️ Bug fix (non-breaking change which fixes an issue)
❌ Breaking change (fix or feature that would cause existing functionality to change)
🧹 Code refactor
✅ Build configuration change
📝 Documentation
🗑️ Chore

3a4oT and others added 7 commits April 25, 2024 14:32
…agsmith#48)

* - Bumped `swift-tools` to swift 5.9.
 - Enabled `StrictConcurrency=complete` mode in order to be prepared for upcoming Swift 6. Implemented `Sendable` support for Flagsmith client and friends.
 - Enabled `ExistentialAny` feature in order to be prepared for upcoming Swift 6. Code adjustments
 - adjusted tests for `StrictConcurrency=complete` mode .

* adjusted macOS runner

* cleanup
Copy link
Contributor

@matthewelwell matthewelwell left a comment

Choose a reason for hiding this comment

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

A few minor comments and I'm still a little uneasy including the whitespace changes in this PR, but the actual logical changes seem sound for the most part.

FlagsmithClient/Classes/Flagsmith.swift Show resolved Hide resolved
FlagsmithClient/Classes/Flagsmith.swift Outdated Show resolved Hide resolved
# Conflicts:
#	Example/FlagsmithClient/AppDelegate.swift
#	FlagsmithClient/Classes/Feature.swift
#	FlagsmithClient/Classes/Flag.swift
#	FlagsmithClient/Classes/Flagsmith.swift
#	FlagsmithClient/Classes/FlagsmithError.swift
#	FlagsmithClient/Classes/Internal/APIManager.swift
#	FlagsmithClient/Classes/Internal/FlagsmithAnalytics.swift
#	FlagsmithClient/Classes/Trait.swift
#	FlagsmithClient/Classes/TypedValue.swift
#	FlagsmithClient/Classes/UnknownTypeValue.swift
#	FlagsmithClient/Tests/APIManagerTests.swift
#	Package.swift
@matthewelwell matthewelwell merged commit a15afa0 into Flagsmith:main May 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants