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

Made the project compatible with Xcode 15.2 #9

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

Conversation

martijnvdwal
Copy link

Hi Uncle Bob!

I am following along on the Mobile App Case Study video's, I saw the GitHub repository linked in the description and had to check it out!

Of course, The current version of Xcode began complaining that Swift 3 is not supported anymore, with no direct migration available. Luckily, only very small changes were needed to migrate from Swift 3 to Swift 5, I also updated some settings per Xcode's recommendation, now the project is entirely free of errors/warnings. I also deleted the UI test target. 😄

I also saw Xcode was compiling that the return value from some functions were not used, to silence this warning it has to be prefixed with _ = , I'm not very fond of it, but I think it beats all the warnings you otherwise would see.

All changes I made

  • Made the source compatible with Xcode 15.2 and Swift 5
  • Resolved all warnings
  • Removed GomokuUITests
  • Fixed incorrect player status label

The application still works on device, and all unit tests pass ✅

Kind regards,
Martijn van der Wal

The player label "updated"  with the player that placed the stone in case the player did not win, now it asks the game whose turn it is and presents that.
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.

1 participant