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

Update for Swift 5.0 Support #430

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

Nomad00
Copy link

@Nomad00 Nomad00 commented May 30, 2019

Update for Swift 5.0 Support

  • Update podspec to set version to 5.0.0.
  • Update podspec to set swift_version to 5.0.
  • Update project targets to have 9.0 Deployment Target (this was already specified in podspec).
  • Update project with Xcode 10.2.1 recommended settings.
  • Refactor uses of fabs(_:) to abs(_:) to resolve warnings.
  • Convert SlideMenuControllerSwift-iOS target to Swift 5.0.0 via wizard (no changes required).
  • Add no-op @unknown default: handling for handleLeftPanGesture(_:) to resolve warnings.
  • Add no-op @unknown default: handling for handleRightPanGesture(_:)to resolve warnings.
  • Convert SlideMenuControllerSwift target to Swift 5.0.0 via wizard (no changes required).
  • Refactor usage of .characters.count to .count to resolve build error.
  • Refactor substring(_:) to use .suffix instead of removed .substring to build error.
  • Update CHANGELOG.md to indicate Swift 5.0 change.
  • Remove deprecated .swift-version file.

Includes: Update for Swift 4.2 support

  • Remove unneeded entry of Info.plist from the Copy Bundle Resources build phase.
  • Apply Xcode suggested project settings updates.
  • Convert SlideMenuControllerSwift scheme to Swift 4.2.
  • Convert SlideMenuControllerSwift-iOS scheme to Swift 4.2.
  • Update podspec to increase version number to 4.2.0.
  • Add swift_version to podspec (4.2).
  • Update CHANGELOG.md to indicate Swift 4.2 change.

- Remove unneeded entry of `Info.plist` from the `Copy Bundle Resources` build phase.
- Apply Xcode suggested project settings updates.
- Convert `SlideMenuControllerSwift` scheme to Swift 4.2.
- Convert `SlideMenuControllerSwift-iOS` scheme to Swift 4.2.
- Update `podspec` to increase version number to `4.2.0`.
- Add `swift_version` to `podspec` (`4.2`).
- Update `CHANGELOG.md` to indicate Swift 4.2 change.
Update for Swift 4.2 support
- Update `podspec` to set `version` to `5.0.0`.
- Update `podspec` to set `swift_version` to `5.0`.
- Update project targets to have `9.0` Deployment Target (this was already specified in `podspec`).
- Update project with Xcode `10.2.1` recommended settings.
- Refactor uses of `fabs(_:)` to `abs(_:)` to resolve warnings.
- Convert `SlideMenuControllerSwift-iOS` target to Swift `5.0.0` via wizard (no changes required).
- Add no-op `@unknown default:` handling for `handleLeftPanGesture(_:)` to resolve warnings.
- Add no-op `@unknown default:` handling for `handleRightPanGesture(_:)`to resolve warnings.
- Convert `SlideMenuControllerSwift` target to Swift `5.0.0` via wizard (no changes required).
- Refactor usage of `.characters.count` to `.count` to resolve build error.
- Refactor `substring(_:)` to use `.suffix` instead of removed `.substring` to build error.
Update for Swift 5.0.0 support
@AlonsoFloo
Copy link

Is possible to accept this pull request ?

@AlonsoFloo AlonsoFloo mentioned this pull request Jul 3, 2019
@Feghal
Copy link

Feghal commented Jul 3, 2019

I made a fork with swift 5 support a while ago, seems they are not accepting any pull request.
https://github.com/AtomicSLLC/SlideMenuControllerSwift

add this to your podfile
pod 'SlideMenuControllerSwift', :git => 'https://github.com/AtomicSLLC/SlideMenuControllerSwift.git', :branch => 'swift5'

@zhouhao27
Copy link

Is this merged to master already?

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.

5 participants