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.

Sign up for GitHub

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

Jump to bottom

Assorted Keychain cleanup #133

Merged
merged 3 commits into from
Feb 10, 2017
Merged

Assorted Keychain cleanup #133

merged 3 commits into from
Feb 10, 2017

Conversation

mattrubin
Copy link
Owner

  • Remove an optional whose only purpose was support for Xcode 7
  • Explicitly clarify which PersistentToken.init is used by the keychain

@mattrubin
Copy link
Owner Author

I wish Codecov recognized when a decrease in coverage percentage was caused simply by deleting covered lines, and not by adding any uncovered lines.
Bypassing Codecov's objections to merge this.

@mattrubin mattrubin merged commit 7a6804d into develop Feb 10, 2017
@mattrubin mattrubin deleted the keychain-cleanup branch February 10, 2017 16:37
@mattrubin mattrubin added this to the 3.0.1 milestone Mar 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.0.1
Development

Successfully merging this pull request may close these issues.
1 participant
@mattrubin