Skip to content

Conversation

@NandanPrabhu
Copy link
Contributor

@NandanPrabhu NandanPrabhu commented May 12, 2025

  • 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

PR has swift 6 changes in the repo and repo targets. this is a followup PR on Xcode 16 support in CI. #275

📎 References

🎯 Testing

@NandanPrabhu NandanPrabhu requested a review from a team as a code owner May 12, 2025 05:57
@NandanPrabhu NandanPrabhu requested a review from Widcket May 12, 2025 05:57
@NandanPrabhu NandanPrabhu changed the title SDK-5963 swift 6 upgrade Swift 6 upgrade May 12, 2025
@Widcket
Copy link
Contributor

Widcket commented May 12, 2025

@NandanPrabhu NandanPrabhu changed the title Swift 6 upgrade Swift 6 upgrade to the repo May 29, 2025
}

extension JWTDecodeError: @retroactive Equatable {}
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.

There is a warning here.

C1896F0F2C1C731000CA1000 /* Quick.xcframework in Frameworks */,
C1896F0C2C1C730F00CA1000 /* Nimble.xcframework in Frameworks */,
C1896F0F2C1C731000CA1000 /* (null) in Frameworks */,
C1896F0C2C1C730F00CA1000 /* (null) in Frameworks */,
Copy link
Contributor

Choose a reason for hiding this comment

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

If you're removing Quick and Nimble, there should be no entries here.

isa = PBXFrameworksBuildPhase;
buildActionMask = 2147483647;
files = (
D41DECE32DC89DE100F5B1A4 /* CwlPreconditionTesting in Frameworks */,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove the CwlPreconditionTesting dependency as well (IIRC, it was added as an SPM dependency).

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