Skip to content
This repository was archived by the owner on Oct 13, 2021. It is now read-only.

Updated to Swift 4, removed some unnecessary code and cleaned the example project #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dejanskledar
Copy link

No description provided.

@dejanskledar
Copy link
Author

@startupthekid Any progress?

@thebarndog
Copy link
Owner

I honestly did not see this until right now...

@thebarndog
Copy link
Owner

Huh well I couldn't find out how to turn on notifs for new issues/PRs but I'll figure that out later. I really didn't even know people used this repository; I came really close to completely deleting it about a month ago.

This PR looks good; unfortunately I made this repo when I was a really green engineer, so no tests or anything. I'm not even sure this is the best way to solve the problem because it involves overriding well-defined behavior.

I'll merge this PR since it can't hurt but I'll try to think of a better to solve this over the next few days.

@thebarndog
Copy link
Owner

Actually one quick thing @dejanskledar, if you wouldn't mind updating the README as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants