Skip to content

Upgrades Xcode version to 16 in CI#275

Merged
NandanPrabhu merged 8 commits intomasterfrom
feature/SDK-5914
Apr 25, 2025
Merged

Upgrades Xcode version to 16 in CI#275
NandanPrabhu merged 8 commits intomasterfrom
feature/SDK-5914

Conversation

@NandanPrabhu
Copy link
Contributor

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

📋 Changes

Upgrades

  • Xcode version to 16.1 in CI
  • Swift versions upgrades in Package.swift , JWTDecode.podspec and Readme.md

📎 References

🎯 Testing

@NandanPrabhu NandanPrabhu requested a review from a team as a code owner April 23, 2025 10:12
Package.swift Outdated
@@ -1,4 +1,4 @@
// swift-tools-version:5.9
// swift-tools-version:6.0.2
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// swift-tools-version:6.0.2
// swift-tools-version:6.0

Any particular reason for specifying 6.0.2 here and in the Podspec?


s.source_files = 'JWTDecode/*.swift'
s.swift_versions = ['5.9']
s.swift_versions = ['6.0.2', '6.0.3', '6.1']
Copy link
Contributor

@Widcket Widcket Apr 23, 2025

Choose a reason for hiding this comment

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

Suggested change
s.swift_versions = ['6.0.2', '6.0.3', '6.1']
s.swift_versions = ['6.0', '6.1']

Any particular reason for specifying 6.0.2 and 6.0.3 here?

#if compiler(>=6.0)
extension JWTDecodeError: @retroactive Equatable {}
#else
extension JWTDecodeError: Equatable {}
Copy link
Contributor

Choose a reason for hiding this comment

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

In this case, the @retroactive declaration should be left in place, because it will always be built with the Swift 6.0 compiler (or later).

Copy link
Contributor

Choose a reason for hiding this comment

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

See the warnings below:

Screenshot 2025-04-23 at 15 01 16

path: "JWTDecodeTests",
exclude: ["Info.plist"])
exclude: ["Info.plist"],
swiftSettings: [.swiftLanguageMode(.v5)])
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!

@NandanPrabhu NandanPrabhu merged commit 017e04b into master Apr 25, 2025
13 checks passed
@NandanPrabhu NandanPrabhu deleted the feature/SDK-5914 branch April 25, 2025 11:37
@Widcket Widcket mentioned this pull request May 12, 2025
2 tasks
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.

2 participants