sonsongithub / reddift

Swift Reddit API Wrapper
MIT License
242 stars 52 forks source link

Swift 5 / Xcode 11 support [WIP] #223

Open jakunico opened 4 years ago

jakunico commented 4 years ago

I had an old project that depended on reddift - I'm trying to resurrect that project and reddift was not working with swift 5 and Xcode 11. This PR should hopefully fix that.

image

App, frameworks and tests compile with no warnings:

image

I've faced a couple of challenges with tests:

  1. Some tests are failing to me, from what I saw some of them depend on your reddit user, for instance UsersTest.testGetUserContents uses sonson_twit username which obviously does not match mine (see image below).
  2. Travis needs to read some encrypted variables which are not available when you are a contributor (as you would be able to edit the travis script and print out those). Maybe you can merge temporarily into a swift-5 branch instead of master and then run tests on travis by opening a swift-5 > master PR.

image