[GitHub] markusthoemmes commented on issue #3355: Add requiredProperties method to ArtifactStore

2018-02-27 Thread GitBox
markusthoemmes commented on issue #3355: Add requiredProperties method to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/issues/3355#issuecomment-368862540 In theory it can but in the past we did all the pureconfig moves by "biting the bullet" and fix every deployment me

[GitHub] markusthoemmes commented on issue #3355: Add requiredProperties method to ArtifactStore

2018-02-27 Thread GitBox
markusthoemmes commented on issue #3355: Add requiredProperties method to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/issues/3355#issuecomment-368861156 Alternatively, we should move the ArtifactStore to `pureconfig` which completely localizes the concern to the imple