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

Drop support for iOS 12, tvOS 12, macOS 10.15, watch0S < 7, and Xcode 13 #184

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

Widcket
Copy link
Contributor

@Widcket Widcket commented Jun 14, 2023

  • 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

This PR raises the deployment target of JWTDecode.swift as follows:

  • iOS 12.0 -> 13.0
  • tvOS 12.0 -> 13.0
  • macOS 10.15 -> 11.0
  • watchOS 6.2 -> 7.0

Additionally, support for Xcode 13 was dropped, and as such the minimum Swift version was raised to 5.7 (the version shipped with Xcode 14.0).

@Widcket Widcket requested a review from a team as a code owner June 14, 2023 16:45
@Widcket Widcket added the review:small Small review label Jun 14, 2023
@Widcket Widcket merged commit 6ee2d30 into master Jun 14, 2023
@Widcket Widcket deleted the chore/drop-support-old-platforms-xcode branch June 14, 2023 17:00
@Widcket Widcket mentioned this pull request Jun 14, 2023
@userNameIsBoris
Copy link

userNameIsBoris commented Jun 15, 2023

Hi! Just wondering, why did you drop it? It doesn't look like necessary change.

I'm working on the project with iOS 12 deployment target and today i ran into error after package update. Had to fix dependency on 1.0.1.

@Widcket
Copy link
Contributor Author

Widcket commented Jun 15, 2023

Hi @userNameIsBoris, we're aligning our Swift libraries to support iOS 13+, as per our Support Policy.

@userNameIsBoris
Copy link

@Widcket Got it, thanks. Have a nice day!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review:small Small review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants