Skip to content

Conversation

@calda
Copy link
Contributor

@calda calda commented Mar 22, 2020

@calda calda force-pushed the master branch 2 times, most recently from f952862 to 7983977 Compare March 23, 2020 14:12
@calda calda force-pushed the master branch 2 times, most recently from 1fd5695 to 668e7b5 Compare March 26, 2020 04:29
@calda
Copy link
Contributor Author

calda commented Aug 12, 2020

@airspeedswift Are there any actionable next-steps that could move this proposal towards a formal review?

@benrimmington benrimmington changed the base branch from master to main November 10, 2020 03:06
@tomerd
Copy link
Contributor

tomerd commented Apr 17, 2021

hi @calda, apologies for the slow response. do you mind launching a new pitch thread on the forums on this idea, as the team has some feedback best given in the context of a pitch.

@calda
Copy link
Contributor Author

calda commented Apr 17, 2021

Thanks for the response @tomerd -- would you mind sharing feedback on the existing thread (https://forums.swift.org/t/codingkeypath-add-support-for-encoding-and-decoding-nested-objects-with-dot-notation/34710/26)?

I don't have any new information to add (that isn't already present in that thread) so I would feel a bit silly creating a new thread from scratch. I can find time to start a new thread if that is your strong preference, though.

@tomerd
Copy link
Contributor

tomerd commented Apr 17, 2021

I don't have any new information to add (that isn't already present in that thread) so I would feel a bit silly creating a new thread from scratch. I can find time to start a new thread if that is your strong preference, though.

that can work too. I will point the folks that had feedback there.

@tomerd
Copy link
Contributor

tomerd commented Aug 12, 2021

@calda given our recent discussion on the pitch thread with @parkera, should we close this PR, or do you want to amend it?

@calda
Copy link
Contributor Author

calda commented Aug 12, 2021

We can close this one. Thanks for your feedback and suggestions!

@calda calda closed this Aug 12, 2021
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