Skip to content

[NSURLQueryItem] NSCoding Implementation #712

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

Merged
merged 1 commit into from
Dec 15, 2016

Conversation

naithar
Copy link
Contributor

@naithar naithar commented Nov 11, 2016

Added NSCoding to NSURLQueryItem.
Added tests.
Overriden isEqual(_:) to conform to MacOS Foundation.

[NSURLQueryItem] Added tests for NSCoding

[NSURLQueryItem] NSCoder implementation

[NSURLQueryItem] 'isEqual' overriden

[Docs] NSURLQueryItem status update
@naithar
Copy link
Contributor Author

naithar commented Nov 18, 2016

@parkera please review

@parkera
Copy link
Contributor

parkera commented Nov 18, 2016

Looks good to me, thanks.

@parkera
Copy link
Contributor

parkera commented Nov 18, 2016

@swift-ci please test and merge

@naithar
Copy link
Contributor Author

naithar commented Nov 18, 2016

Jenkins showed that it failed with: error: use of undeclared identifier 'TimePoint' TimePoint<> LastAccessTime; if i'm not mistaken. It's not PR issue, is it?

@parkera
Copy link
Contributor

parkera commented Nov 18, 2016

I think that master is busted; I saw that same failure locally.

@pushkarnk
Copy link
Member

@swift-ci please test

@naithar
Copy link
Contributor Author

naithar commented Dec 14, 2016

FAIL: kqtest doesn't seem to be PR issue. Could it be retested please?

@parkera
Copy link
Contributor

parkera commented Dec 14, 2016

@swift-ci please test

@naithar
Copy link
Contributor Author

naithar commented Dec 15, 2016

Looks like the last testing performed correctly.

@parkera
Copy link
Contributor

parkera commented Dec 15, 2016

@swift-ci please test and merge

@swift-ci swift-ci merged commit c5498b3 into swiftlang:master Dec 15, 2016
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