Skip to content

Missing capitalisation of 'must'#968

Merged
webron merged 1 commit intoOAI:OpenAPI.nextfrom
MikeRalphson:patch-6
Mar 27, 2017
Merged

Missing capitalisation of 'must'#968
webron merged 1 commit intoOAI:OpenAPI.nextfrom
MikeRalphson:patch-6

Conversation

@MikeRalphson
Copy link
Copy Markdown
Member

No description provided.

@ePaul
Copy link
Copy Markdown
Contributor

ePaul commented Mar 7, 2017

If you are at it, could you fix the other occurrence of must in the document (in the OpenAPI object's field table, tags).

@MikeRalphson
Copy link
Copy Markdown
Member Author

@ePaul if you mean

Not all tags that are used by the Operation Object must be declared

I think that "not all .. must be declared" isn't a case of MUST or MUST NOT. But it may be better to reword this as

Not all tags that are used by the Operation Object need to be declared

to avoid the ambiguity.

@ePaul
Copy link
Copy Markdown
Contributor

ePaul commented Mar 7, 2017

You are right, I somehow didn't read the full sentence. Your rewording seems fine.

@MikeRalphson
Copy link
Copy Markdown
Member Author

Happy to fold in the other change if required.

@fehguy fehguy closed this Mar 7, 2017
@webron webron reopened this Mar 7, 2017
@webron webron merged commit f22129c into OAI:OpenAPI.next Mar 27, 2017
@handrews handrews added this to the v3.0.0-rc1 milestone Feb 8, 2024
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.

5 participants