Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Fix SwiftLint Errors #9
Fix SwiftLint Errors #9
Changes from 8 commits
e0e4130
12e1ac7
a723b27
710d707
600fa81
d774249
e527903
05a86e7
378248c
7399d37
f866536
15c1d9a
e6aeec0
f487baf
868ec78
f2e18e3
dca1bc2
fed6eb7
e776abb
fe0d859
47c0932
3b81390
3ba6ee4
9eb914e
5927703
5899b5a
1eee004
5310345
18aae3e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why adding this implementation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added one subscript and two functions to set and get an [Any] to subscript in KeychainStore for pass
testSetComplextObject
Test Cases. But I would love to get your feedback. Thank youThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be no need for that, as a T can be an [U] if you specify it so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for advice me. I removed these implement.
I got an error message below
So I removed
where T: Decodable
where T: Codable
where T: Encodable
Is it ok? It works fine..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not OK, as we do not want at compile time to store non Codable entries.
Do we really need array support?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we don't need [Any] object support.