You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openwhisk.apache.org by gi...@git.apache.org on 2017/08/23 11:15:50 UTC

[GitHub] mrutkows opened a new issue #376: Why default version to "master" inside manifest parser

mrutkows opened a new issue #376: Why default version to "master" inside manifest parser
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/376
 
 
   in manifest_parser.yaml we default to version = "master"; however, this has no meaning in the specification.  We need to examine why this was done and if we can remove it.
   
   for reference, it is set as follows:
   ```
   func (dm *YAMLParser) ComposeDependencies(mani *ManifestYAML, projectPath string) (map[string]utils.DependencyRecord, error) {
   
   	var errorParser error
   	depMap := make(map[string]utils.DependencyRecord)
   	for key, dependency := range mani.Package.Dependencies {
   		version := dependency.Version
   		if version == "" {
   			version = "master"
   		}
   ```
 
----------------------------------------------------------------
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