Skip to content

Conversation

@Woildan
Copy link
Contributor

@Woildan Woildan commented Oct 11, 2018

I needed the fix for #1325 so I took the liberty to use @RyanBertrand's branch and submit it in this PR.

All I did is fix the path of a renamed file in the xcodeproj, change some comments and rewrite the commits history.

I'm not familiar with the development procedure of the Parse SDK, but it builds on my machine, tests are green and the SDK works in my app.

@Woildan Woildan force-pushed the rename-PFFile-and-PFLogger branch from 69a8ebd to e607a36 Compare October 11, 2018 22:39
@flovilmart
Copy link
Contributor

Can you check the tests that failed?

@Woildan
Copy link
Contributor Author

Woildan commented Oct 12, 2018

Tests that failed are KeychainStoreTests.
I don't think this has anything to do with my PR. Possibly fixed by #1353

@Woildan Woildan closed this Oct 12, 2018
@Woildan Woildan deleted the rename-PFFile-and-PFLogger branch October 12, 2018 09:08
@Woildan Woildan restored the rename-PFFile-and-PFLogger branch October 12, 2018 09:08
@Woildan Woildan reopened this Oct 12, 2018
@flovilmart flovilmart merged commit 067ede1 into parse-community:master Oct 24, 2018
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.

3 participants