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 to latest Pods and fixed crashes from forced unwraps #48

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

Conversation

imaczealot
Copy link

Update Pods
Fix crashes
Swift code clean up
Add print statements useful for learning the project
Recommend not tracking (including in git) all files from Pods directory

Fix crashes
Add print statements useful for learning the project
Recommend not tracking (including in git) all files from Pods directory
@imaczealot
Copy link
Author

I ran into many crashes evaluating the code. I fixed several and did some code clean up.

I'd recommend removing some of the files that are being tracked. I've added a standard .gitignore file to help, but all existing tracked files will need to be removed from git cache.

@imaczealot imaczealot closed this Sep 10, 2018
@imaczealot imaczealot reopened this Sep 11, 2018
@imaczealot
Copy link
Author

Doh! DIdn't mean to close this PR.

@sf-jed-kyung
Copy link
Contributor

@imaczealot Thank you for your interest. We are preparing the new version of this project. The new project will have a new UI, new structures, and more features. That's why I'm not able to merge your PR now. But, I'll adopt many things from your PR and apply them to the new version. They will be very helpful for the new one and I hope we can release it soon. Thanks again for your interest in this project.

@imaczealot
Copy link
Author

That's great! Can you give a timeframe for when the next release will be posted?

@ezet
Copy link

ezet commented Oct 3, 2018

We are also interested in a timeframe for the updated version?

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.

4 participants