Skip to content

*: Replace "array" type with "array of objects" - #749

Merged
mrunalp merged 1 commit into
opencontainers:masterfrom
wking:array-of-x
Mar 30, 2017
Merged

*: Replace "array" type with "array of objects"#749
mrunalp merged 1 commit into
opencontainers:masterfrom
wking:array-of-x

Conversation

@wking

Copy link
Copy Markdown
Contributor

We have a few different element types in our arrays, so it's useful to clarify the element type for the property being specified. Before this commit:

$ sed -n 's|.*\*\*`\([^`]*\)`\*\*[^(]*(\([^,]*\),.*|\2|p' *.md | sort | uniq -c | grep array
7 array
1 array of ints
8 array of objects
13 array of strings

All of the bare array instances turned out to be arrays of objects.

We have a few different element types in our arrays, so it's useful to
clarify the element type for the property being specified. Before
this commit:
$ sed -n 's|.*\*\*`\([^`]*\)`\*\*[^(]*(\([^,]*\),.*|\2|p' *.md | sort | uniq -c | grep array
7 array
1 array of ints
8 array of objects
13 array of strings
All of the bare 'array' instances turned out to be arrays of objects.
Signed-off-by: W. Trevor King <wking@tremily.us>
@hqhq

hqhq commented Mar 30, 2017

Copy link
Copy Markdown
Contributor

LGTM

Approved with PullApprove

1 similar comment
@mrunalp

mrunalp commented Mar 30, 2017

Copy link
Copy Markdown
Contributor

LGTM

Approved with PullApprove

@mrunalp
mrunalp merged commit d094a5c into opencontainers:masterMar 30, 2017
@wking
wking deleted the array-of-x branch April 5, 2017 16:19
@vbattsvbatts mentioned this pull request Jul 5, 2017
Sign up for freeto 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.

3 participants

@wking@hqhq@mrunalp