[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370113952 I think I'm ready for review and final merge of this PR This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370113609 > With now 4.2 announced 4.1 is stable, so skip/remove 4.0 support only do 4.1 for now Going to move this to a new issue https://github.com/apache/incubator-openwhisk-runtime-swift/issues/30 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370027301 Thanks @jthomas for the review and starting the work to externalize the epilogue in to it's own library/package. >Support @escaping closures to allow users to use with async calls. ``` func _run_main(mainFunction: (In, @escaping (Out?, Error?) -> Void) -> Void) { } ``` This just needs the annotation adding and doesn't affect existing code or people who don't use that. Guess what I have in my TODOs? ### TODOS: - [x] Finish with the Codable test cases - [x] Use `@escaping` instead for the callback - [x] Refresh again with latest 4.1 snapshot - [x] With now 4.2 announced 4.1 is stable, so skip/remove 4.0 support only do 4.1 for now >https://github.com/jthomas/OpenWhiskAction/blob/swift4/Sources/epilogue.swift#L84-L93 This helps the user understand when there has been problems using the Codable API versus something else. I will add the one for more verbose errors ? - [x] Improved error handled to give more descriptive messages, e.g. >Sync method style APIs should probably support throws. This enables uses to use normal error handling to surface issues. I thought about this one, will look into this but probably a different PR since it's an improvement to dictionary sync that we have today. https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29 Super happy that someone is trying this stuff ? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370113363 Woot ! - [x] Improved error handled to give more descriptive messages, e.g. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370113363 Woot ! - [ ] Improved error handled to give more descriptive messages, e.g. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370027301 Thanks @jthomas for the review and starting the work to externalize the epilogue in to it's own library/package. >Support @escaping closures to allow users to use with async calls. ``` func _run_main(mainFunction: (In, @escaping (Out?, Error?) -> Void) -> Void) { } ``` This just needs the annotation adding and doesn't affect existing code or people who don't use that. Guess what I have in my TODOs? ### TODOS: - [x] Finish with the Codable test cases - [x] Use `@escaping` instead for the callback - [x] Refresh again with latest 4.1 snapshot - [ ] With now 4.2 announced 4.1 is stable, so skip/remove 4.0 support only do 4.1 for now >https://github.com/jthomas/OpenWhiskAction/blob/swift4/Sources/epilogue.swift#L84-L93 This helps the user understand when there has been problems using the Codable API versus something else. I will add the one for more verbose errors ? - [x] Improved error handled to give more descriptive messages, e.g. >Sync method style APIs should probably support throws. This enables uses to use normal error handling to surface issues. I thought about this one, will look into this but probably a different PR since it's an improvement to dictionary sync that we have today. https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29 Super happy that someone is trying this stuff ? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370058906 Woot ! - [x] Use @escaping instead for the callback This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370058906 Woot ! - [ ] Use @escaping instead for the callback This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370058906 Woot ! - [x] Use `@escaping` instead for the callback This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370027301 Thanks @jthomas for the review and starting the work to externalize the epilogue in to it's own library/package. >Support @escaping closures to allow users to use with async calls. ``` func _run_main(mainFunction: (In, @escaping (Out?, Error?) -> Void) -> Void) { } ``` This just needs the annotation adding and doesn't affect existing code or people who don't use that. Guess what I have in my TODOs? ### TODOS: - [x] Finish with the Codable test cases - [x] Use `@escaping` instead for the callback - [x] Refresh again with latest 4.1 snapshot - [ ] With now 4.2 announced 4.1 is stable, so skip/remove 4.0 support only do 4.1 for now >https://github.com/jthomas/OpenWhiskAction/blob/swift4/Sources/epilogue.swift#L84-L93 This helps the user understand when there has been problems using the Codable API versus something else. I will add the one for more verbose errors ? - [ ] Improved error handled to give more descriptive messages, e.g. >Sync method style APIs should probably support throws. This enables uses to use normal error handling to surface issues. I thought about this one, will look into this but probably a different PR since it's an improvement to dictionary sync that we have today. https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29 Super happy that someone is trying this stuff ? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370057360 Woot ! - [ ] Refresh again with latest 4.1 snapshot This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370056572 Woot ! - [ ] Finish with the Codable test cases This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370056572 Woot ! - [x] Finish with the Codable test cases This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370056572 Woot ! - [ ] Finish with the Codable test cases This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370027301 Thanks @jthomas for the review and starting the work to externalize the epilogue in to it's own library/package. >Support @escaping closures to allow users to use with async calls. ``` func _run_main(mainFunction: (In, @escaping (Out?, Error?) -> Void) -> Void) { } ``` This just needs the annotation adding and doesn't affect existing code or people who don't use that. Guess what I have in my TODOs? ### TODOS: - [x] Finish with the Codable test cases - [ ] Use `@escaping` instead for the callback - [ ] Refresh again with latest 4.1 snapshot - [ ] With now 4.2 announced 4.1 is stable, so skip/remove 4.0 support only do 4.1 for now >https://github.com/jthomas/OpenWhiskAction/blob/swift4/Sources/epilogue.swift#L84-L93 This helps the user understand when there has been problems using the Codable API versus something else. I will add the one for more verbose errors ? - [ ] Improved error handled to give more descriptive messages, e.g. >Sync method style APIs should probably support throws. This enables uses to use normal error handling to surface issues. I thought about this one, will look into this but probably a different PR since it's an improvement to dictionary sync that we have today. https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29 Super happy that someone is trying this stuff ? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-370027301 Thanks @jthomas for the review and starting the work to externalize the epilogue in to it's own library/package. >Support @escaping closures to allow users to use with async calls. ``` func _run_main(mainFunction: (In, @escaping (Out?, Error?) -> Void) -> Void) { } ``` This just needs the annotation adding and doesn't affect existing code or people who don't use that. Guess what I have in my TODOs? ### TODOS: - [ ] Finish with the Codable test cases - [ ] Use `@escaping` instead for the callback - [ ] Refresh again with latest 4.1 snapshot - [ ] With now 4.2 announced 4.1 is stable, so skip/remove 4.0 support only do 4.1 for now >https://github.com/jthomas/OpenWhiskAction/blob/swift4/Sources/epilogue.swift#L84-L93 This helps the user understand when there has been problems using the Codable API versus something else. I will add the one for more verbose errors ? - [ ] Improved error handled to give more descriptive messages, e.g. >Sync method style APIs should probably support throws. This enables uses to use normal error handling to surface issues. I thought about this one, will look into this but probably a different PR since it's an improvement to dictionary sync that we have today. https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29 Super happy that someone is trying this stuff ? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-365519693 TODOs - [x] Handle when input params is empty - [x] Add tests cases for Codable This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-365519693 TODOs - [x] Handle when input params is empty - [ ] Add tests cases for Codable This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-365519693 TODOs - [ ] Handle when input params is empty - [ ] Add tests cases for Codable This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] csantanapr commented on issue #23: Add Codable Support for Swift 4.x
csantanapr commented on issue #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#issuecomment-365519578 Depends on #22 Closes #2 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services