Skip to content

[WIP] Combine AnyDecodable + AnyEncodable into AnyCodable#2770

Closed
NachoSoto wants to merge 5 commits into
mainfrom
any-codable
Closed

[WIP] Combine AnyDecodable + AnyEncodable into AnyCodable#2770
NachoSoto wants to merge 5 commits into
mainfrom
any-codable

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

Follow up to #2769.

@NachoSoto NachoSoto requested a review from a team July 7, 2023 22:20
@NachoSoto NachoSoto changed the title [WIP] Combine AnyDecodable + AnyEncodable into AnyCodable Combine AnyDecodable + AnyEncodable into AnyCodable Jul 10, 2023
@NachoSoto NachoSoto marked this pull request as ready for review July 10, 2023 15:16
@NachoSoto

Copy link
Copy Markdown
Contributor Author

TODO: verify that nested dictionaries don't have AnyCodable types inside

@NachoSoto

Copy link
Copy Markdown
Contributor Author

I'm considering closing this, it makes things unnecessarily more complex.

@NachoSoto NachoSoto marked this pull request as draft July 11, 2023 00:05
@NachoSoto NachoSoto changed the title Combine AnyDecodable + AnyEncodable into AnyCodable [WIP] Combine AnyDecodable + AnyEncodable into AnyCodable Jul 11, 2023
@NachoSoto NachoSoto closed this Aug 11, 2023
@NachoSoto NachoSoto deleted the any-codable branch August 11, 2023 19:28
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.

2 participants