[Issue #77] Clean up platform specific logic and remove unnecessary f… #82
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…iles
Modify all of the os() compiler directives to just make an exception for linux, but group other platforms together
Specify platform versions in Package.swift to match those in cocoapods
Description
Extracted some of the changes from #78 that are just around cleaning up the compiler directives around Linux vs Apple platforms.
Also cleaned up any around Swift version being >=5, and instead have the dependency managers (cocoapods or SPM) enforce already being on Swift 5+.
No logic changes - only restructured the "#if" logic to be a little cleaner, and should be able to easily support VisionOS, iPadOs, or other future Apple platforms without a code change - just a Package.swift or podspec change.
After Merging
After merging this in, could consider creating a new tag - 2.0.3.
Motivation and Context
This is Part 1 of resolving Issue #77
Trying to separate out the changes to make them easier to review. (from the discussion on the Pull Request #78 )
How Has This Been Tested?
Tested locally via unit testing and in our app that consumes this via SwiftJWT.
Checklist: