Merge lp://staging/~jtv/gwacl/document-slots into lp://staging/gwacl

Proposed by Jeroen T. Vermeulen
Status: Merged
Approved by: Jeroen T. Vermeulen
Approved revision: 168
Merged at revision: 168
Proposed branch: lp://staging/~jtv/gwacl/document-slots
Merge into: lp://staging/gwacl
Diff against target: 17 lines (+4/-3)
1 file modified
xmlobjects.go (+4/-3)
To merge this branch: bzr merge lp://staging/~jtv/gwacl/document-slots
Reviewer Review Type Date Requested Status
Julian Edwards (community) Approve
Review via email: mp+173634@code.staging.launchpad.net

Commit message

Document the accepted values for DeploymentSlot. I'm not going as far as enforcing them; we leave that up to Azure. But this should help make it clearer what the field means. I did this after seeing "production" abbreviated to "prod" somewhere and being unsure which way to spell it for Azure.

Description of the change

And gofmt decided to re-align the struct, too.

Jeroen

To post a comment you must log in.
Revision history for this message
Julian Edwards (julian-edwards) :
review: Approve
Revision history for this message
Julian Edwards (julian-edwards) wrote :

It got reformatted because go doc doesn't understand comments in the middle of the struct.

Revision history for this message
Julian Edwards (julian-edwards) wrote :

go fmt, I mean.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches