Skip to content

Commit

Permalink
Change _artifactsLocationSasToken defaultValue to ""
Browse files Browse the repository at this point in the history
  • Loading branch information
philon-msft committed Jun 22, 2016
1 parent f233021 commit 47203ee
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions 1-CONTRIBUTION-GUIDE/best-practices.md
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ When samples contain scripts, templates or other artifacts that need to be made
First, define two standard parameters:

* _artifactsLocation - this is the base URI where all artifacts for the deployment will be staged. The default value should be the samples folder so that the sample can be easily deployed in scenarios where a private location is not required.
* _artifactsocationSasToken - this is the sasToken required to access _artifactsLocation. The default value should be "?" for scenarios where the _artifactsLocation is not secured, for example, the raw GitHub URI.
* _artifactsocationSasToken - this is the sasToken required to access _artifactsLocation. The default value should be "" for scenarios where the _artifactsLocation is not secured, for example, the raw GitHub URI.

```
"parameters": {
Expand All @@ -237,7 +237,7 @@ First, define two standard parameters:
"metadata": {
"description": "The sasToken required to access _artifactsLocation. When the template is deployed using the accompanying scripts, a sasToken will be automatically generated."
},
"defaultValue": "?"
"defaultValue": ""
}
},
Expand Down
2 changes: 1 addition & 1 deletion 201-vm-custom-script-windows/azuredeploy.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
"metadata": {
"description": "The sasToken required to access _artifactsLocation. When the template is deployed using the accompanying scripts, a sasToken will be automatically generated."
},
"defaultValue": "?"
"defaultValue": ""
}
},
"variables": {
Expand Down

0 comments on commit 47203ee

Please sign in to comment.