Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle array examples; document Example usage #1807

Merged
merged 3 commits into from
Dec 1, 2018

Conversation

leon-barrett
Copy link
Contributor

I previously added JSON parsing for examples on object types; this adds
it for array types. It also adds basic documentation for Example.

@@ -74,6 +75,7 @@ type User struct {
// the email address for this user
//
// required: true
// example: user@provider.net
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How come this alignment is off ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, accidentally made spaces instead of a tab. (Elsewhere in the file spaces are used, so it confused my editor.) Fixed.

Leon Barrett added 3 commits November 30, 2018 11:19
I previously added JSON parsing for examples on object types; this adds
it for array types. It also adds basic documentation for Example.

Signed-off-by: Leon Barrett <leon@cogitai.com>
Signed-off-by: Leon Barrett <leon@cogitai.com>
Signed-off-by: Leon Barrett <leon@cogitai.com>
@leon-barrett
Copy link
Contributor Author

@casualjim Thanks. Is there anything else I need to do? (I can't merge. Do I need to figure out commit signing first?)

@fredbi
Copy link
Contributor

fredbi commented Dec 1, 2018

Yes all commits must be pgp signed

@casualjim casualjim merged commit 8b824dd into go-swagger:master Dec 1, 2018
@leon-barrett leon-barrett deleted the leon/example-doc branch December 3, 2018 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants