[GitHub] allen-servedio commented on issue #729: Remove empty maps/interfaces from "export" manifest yaml

[GitBox <gi...@apache.org>]

allen-servedio commented on issue #729: Remove empty maps/interfaces from "export" manifest yaml
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/729#issuecomment-365273354
 
 
   Just another comment on this... It is very unclear at what level you are supposed to define things - esp. since it appears multiple levels will work.
   
   This works:
   
   ```
   project:
     name: MyProject
     namespace: org_space
     version: 0.0.1
     packages:
       rediskv:
         license: Apache-2.0
         inputs: 
           redisURL:
             type: string
             description: Redis database URL
         actions: 
           kvget:
             function: retrieve/index.js
             runtime: nodejs
           kvput:
             function: store/index.js
             runtime: nodejs
   ```
   
   However, this does NOT see the package or action:
   
   ```
   project:
     name: MyProject
     namespace: org_space
     version: 0.0.1
     package:
       name: rediskv
       license: Apache-2.0
       inputs: 
         redisURL:
           type: string
           description: Redis database URL
       actions: 
         kvget:
           function: retrieve/index.js
           runtime: nodejs
         kvput:
           function: store/index.js
           runtime: nodejs
   ```
   
   But, if you change the indention level it DOES see the package and actions:
   
   ```
   project:
     name: MyProject
     namespace: org_space
     version: 0.0.1
   package:
     name: rediskv
     license: Apache-2.0
     inputs: 
       redisURL:
         type: string
         description: Redis database URL
     actions: 
       kvget:
         function: retrieve/index.js
         runtime: nodejs
       kvput:
         function: store/index.js
         runtime: nodejs
   ```
   
   This this also sees the package and actions:
   
   ```
   project:
     name: MyProject
     namespace: org_space
     version: 0.0.1
   packages:
     rediskv:
       license: Apache-2.0
       inputs: 
         redisURL:
           type: string
           description: Redis database URL
       actions: 
         kvget:
           function: retrieve/index.js
           runtime: nodejs
         kvput:
           function: store/index.js
           runtime: nodejs
   ```
   
   So anything that can be done to clearly define how these are supposed to be put together would really help.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services